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.

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

Fixes a bug in AnalyzeRequest.toXContent() #42795

Merged
merged 1 commit into from Jun 4, 2019

Conversation

@romseygeek
Copy link
Contributor

commented Jun 3, 2019

Fix for #39670, for 7.2 branch only as this has been fixed separately in 7.3 by the refactoring in #42197

@elasticmachine

This comment has been minimized.

Copy link
Collaborator

commented Jun 3, 2019

@elasticmachine

This comment has been minimized.

Copy link
Collaborator

commented Jun 3, 2019

@cbuescher cbuescher requested review from cbuescher and removed request for cbuescher Jun 3, 2019

@javanna
javanna approved these changes Jun 4, 2019

@romseygeek romseygeek merged commit c311c91 into elastic:7.2 Jun 4, 2019

8 checks passed

CLA All commits in pull request signed
Details
elasticsearch-ci/1 Build finished.
Details
elasticsearch-ci/2 Build finished.
Details
elasticsearch-ci/bwc Build finished.
Details
elasticsearch-ci/default-distro Build finished.
Details
elasticsearch-ci/docbldesx Build finished.
Details
elasticsearch-ci/oss-distro-docs Build finished.
Details
elasticsearch-ci/packaging-sample Build finished.
Details

@romseygeek romseygeek deleted the romseygeek:7.2-custom-analyze-api-fix branch Jun 4, 2019

romseygeek added a commit that referenced this pull request Jun 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.