Skip to content

Fix maxCompletedTasks parameter in OverlordClientImpl.#14667

Merged
gianm merged 1 commit intoapache:masterfrom
gianm:fix-oci-max-completed-tasks
Jul 26, 2023
Merged

Fix maxCompletedTasks parameter in OverlordClientImpl.#14667
gianm merged 1 commit intoapache:masterfrom
gianm:fix-oci-max-completed-tasks

Conversation

@gianm
Copy link
Copy Markdown
Contributor

@gianm gianm commented Jul 26, 2023

It was sent to the server as "maxCompletedTasks", but the server expects "max". This caused it to be ignored. This bug was introduced in #14581.

It was sent to the server as "maxCompletedTasks", but the server expects
"max". This caused it to be ignored. This bug was introduced in apache#14581.
@gianm gianm added the Bug label Jul 26, 2023
Copy link
Copy Markdown
Contributor

@kfaraz kfaraz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@gianm gianm merged commit 4a68f8a into apache:master Jul 26, 2023
@gianm gianm deleted the fix-oci-max-completed-tasks branch July 26, 2023 22:12
@LakshSingla LakshSingla added this to the 28.0 milestone Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants