Skip to content

Read hits.total.value to get number of hits#15728

Merged
andrewkroh merged 1 commit intoelastic:feature/python3from
andrewkroh:feature/python3-test-mangement
Jan 22, 2020
Merged

Read hits.total.value to get number of hits#15728
andrewkroh merged 1 commit intoelastic:feature/python3from
andrewkroh:feature/python3-test-mangement

Conversation

@andrewkroh
Copy link
Copy Markdown
Member

This should fix test_management.py under Python3. An object was being returned but
the test was expecting to get a number. The format of the Elasticsearch response changed
for the "track_total_hits" enhancement, but this wasn't updated.

This should fix test_management.py under Python3. An object was being returned but
the test was expecting to get a number. The format of the Elasticsearch response changed
for the "track_total_hits" enhancement, but this wasn't updated.
@jsoriano jsoriano mentioned this pull request Jan 22, 2020
63 tasks
Copy link
Copy Markdown
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

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

Thanks! x-pack libbeat build goes to green with this change 🚀

@andrewkroh andrewkroh merged commit 1ceabf0 into elastic:feature/python3 Jan 22, 2020
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
This should fix test_management.py under Python3. An object was being returned but
the test was expecting to get a number. The format of the Elasticsearch response changed
for the "track_total_hits" enhancement, but this wasn't updated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants