Skip to content

fix cannot resolve param at OverlordResource#getTasks#6679

Merged
asdf2014 merged 1 commit intoapache:masterfrom
QiuMM:fix_OverlordResource
Nov 29, 2018
Merged

fix cannot resolve param at OverlordResource#getTasks#6679
asdf2014 merged 1 commit intoapache:masterfrom
QiuMM:fix_OverlordResource

Conversation

@QiuMM
Copy link
Copy Markdown
Member

@QiuMM QiuMM commented Nov 28, 2018

There is no a path param for this endpoint, it should be fixed at #6593, but it was introduced again by #6510.

Copy link
Copy Markdown
Contributor

@jihoonson jihoonson 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 catching this!

@jihoonson jihoonson added the Bug label Nov 28, 2018
Copy link
Copy Markdown
Member

@asdf2014 asdf2014 left a comment

Choose a reason for hiding this comment

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

LGTM

@asdf2014 asdf2014 merged commit c81cb94 into apache:master Nov 29, 2018
@jon-wei jon-wei added this to the 0.14.0 milestone Feb 20, 2019
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.

4 participants