-
Notifications
You must be signed in to change notification settings - Fork 25.2k
fix a wrong guide tip #46164
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
fix a wrong guide tip #46164
Conversation
`builder.getOutputStream().toString()` will do the trick while `builder.string()` only prints the object reference (kind of address)
Pinging @elastic/es-docs |
Pinging @elastic/es-core-features |
Hi @Hearen, thanks for spotting this, you are right, the |
@Hearen thanks for the update, I will run our test suite (although this is a small docs patch) before merging. |
@elasticmachine update branch |
@elasticmachine test this please |
The failing docs checks cannot be run on 5.x because of some CI build changes that haven't been backported. Merging regardless since this change only concerns a docs code block that should not be run. |
builder.getOutputStream().toString()
will do the trick whilebuilder.string()
only prints the object reference (kind of address like"org.elasticsearch.common.xcontent.XContentBuilder@66e889df"
)