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

[docs] fix put_mapping snippet #28814

Merged
merged 1 commit into from
Mar 1, 2018
Merged

Conversation

olcbean
Copy link
Contributor

@olcbean olcbean commented Feb 24, 2018

Add a java snippet to run in an itegration test
in order to guarantee that the snippet is correct

Closes #28778

Add a java snippet to be run in an integration test
in order to guarantee that the snippet is correct

Closes elastic#28778
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

1 similar comment
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@javanna
Copy link
Member

javanna commented Feb 28, 2018

test this please

@javanna javanna merged commit 472acf7 into elastic:master Mar 1, 2018
@javanna
Copy link
Member

javanna commented Mar 1, 2018

thanks @olcbean !

javanna pushed a commit that referenced this pull request Mar 1, 2018
Add a java snippet to be run in an integration test
in order to guarantee that the snippet is correct

Closes #28778
sebasjm pushed a commit to sebasjm/elasticsearch that referenced this pull request Mar 10, 2018
Add a java snippet to be run in an integration test
in order to guarantee that the snippet is correct

Closes elastic#28778
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants