Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

High-level client: list tasks failure to not lose nodeId #31001

Merged
merged 2 commits into from
Jun 1, 2018
Merged

High-level client: list tasks failure to not lose nodeId #31001

merged 2 commits into from
Jun 1, 2018

Commits on May 31, 2018

  1. High-level client: list tasks failure to not lose nodeId 

    This commit reworks testing for `ListTasksResponse` so that random
fields insertion can be tested and xcontent equivalence can be checked
too. Proper exclusions need to be configured, and failures need to be
tested separately. This helped finding a little problem, whenever there
is a node failure returned, the nodeId was lost as it was never printed
out as part of the exception toXContent.
    @javanna
    javanna committed May 31, 2018
    Configuration menu
    Copy the full SHA
    2c709ed View commit details
    Browse the repository at this point in the history

  2. added comment

    @javanna
    javanna committed May 31, 2018
    Configuration menu
    Copy the full SHA
    3ed1829 View commit details
    Browse the repository at this point in the history