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

the code with title PUT MAPPING is wrong at Java API [6.2] » Java API Administration » Indices Administration #28778

Closed
huangRZ11 opened this issue Feb 22, 2018 · 4 comments
Labels
:Core/Infra/Transport API Transport client API >docs General docs changes feedback_needed

Comments

@huangRZ11
Copy link

Describe the feature:

Elasticsearch version (bin/elasticsearch --version):

Plugins installed: []

JVM version (java -version):

OS version (uname -a if on a Unix-like system):

Description of the problem including expected versus actual behavior:

Steps to reproduce:

Please include a minimal but complete recreation of the problem, including
(e.g.) index creation, mappings, settings, query etc. The easier you make for
us to reproduce it, the more likely that somebody will take the time to look at it.

Provide logs (if relevant):

@DaveCTurner DaveCTurner added the >docs General docs changes label Feb 22, 2018
@DaveCTurner
Copy link
Contributor

Please follow the instructions for reporting issues (quoted above) - a two line issue title and no further detail isn't helpful. We need to know a bit more about what you mean by "wrong": what did you see when you tried it, and how does it differ from what you expected? Did you try any variations and did they help?

@DaveCTurner
Copy link
Contributor

Sorry, forgot to mention @elastic/es-clients for a ping.

@DaveCTurner DaveCTurner added the :Core/Infra/Transport API Transport client API label Feb 22, 2018
@clintongormley
Copy link
Contributor

@DaveCTurner btw it should be @elastic/es-core-infra , not the clients team.

@huangRZ11 is referring to the code here: https://www.elastic.co/guide/en/elasticsearch/client/java-api/current/java-admin-indices.html#java-admin-indices-put-mapping (but I don't know what is wrong either).

@olcbean
Copy link
Contributor

olcbean commented Feb 23, 2018

@huangRZ11 do you mean that the example should be :

client.admin().indices().prepareCreate("twitter")   
        .addMapping(
                "tweet", "{\n" +               
                "    \"properties\": {\n" +
...

instead of

client.admin().indices().prepareCreate("twitter")   
        .addMapping("tweet", "{\n" +               
                "    \"tweet\": {\n" +
                "      \"properties\": {\n" +
....

olcbean added a commit to olcbean/elasticsearch that referenced this issue Feb 24, 2018
Add a java snippet to be run in an integration test
in order to guarantee that the snippet is correct

Closes elastic#28778
javanna pushed a commit that referenced this issue 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
javanna pushed a commit that referenced this issue 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 issue 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
Labels
:Core/Infra/Transport API Transport client API >docs General docs changes feedback_needed
Projects
None yet
Development

No branches or pull requests

4 participants