-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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] Update shared attributes for Elasticsearch #25479
Conversation
@debadair If we eventually want to use the shared attributes from books other than the Elasticsearch Reference (e.g. from the Painless and client books), we can move the "include::{docs-dir}/shared/attributes.asciidoc[]" from index-shared1.asciidoc to Versions.asciidoc. I haven't done any work on those other books at this point. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any changes to Versions.asciidoc will impact the other books in the ES project. It's a silent failure when an attribute is undefined, so the new shared file should definitely be included at the Versions.asciidoc level.
BTW, I'd be inclined to pull the attributes for the definitive guide and painless and plugins docs into the shared file, as well. |
20f4f2e
to
0614e25
Compare
0614e25
to
95926c0
Compare
@debadair verified that the other books were okay to use the shared attributes, so I moved the reference to the elastic/docs/shared/attributes.asciidoc from index-shared1.asciidoc to Versions.asciidoc. I also moved the definitive guide, painless, and plugins doc URLs to the shared file too. |
LGTM |
* [DOCS] Update shared attributes for Elasticsearch * [DOCS] Moved shared attributes to Versions.asciidoc * [DOCS] More more book URLs to shared attributes
* [DOCS] Update shared attributes for Elasticsearch * [DOCS] Moved shared attributes to Versions.asciidoc * [DOCS] More more book URLs to shared attributes
Thanks! I've retrofitted to 5.x and 5.5 |
* master: (129 commits) Add doc note regarding explicit publish host Fix typo in name of test Add additional test for sequence-number recovery WrapperQueryBuilder should also rewrite the parsed query. Remove dead code and stale Javadoc Update defaults in documentation (#25483) [DOCS] Add docs-dir to Painless (#25482) Add concurrent deprecation logger test [DOCS] Update shared attributes for Elasticsearch (#25479) Use LRU set to reduce repeat deprecation messages Add NioTransport threads to thread name checks (#25477) Add shortcut for AbstractQueryBuilder.parseInnerQueryBuilder to QueryShardContext Prevent channel enqueue after selector close (#25478) Fix Java 9 compilation issue Remove unregistered `transport.netty.*` settings (#25476) Handle ping correctly in NioTransport (#25462) Tests: Remove platform specific assertion in NioSocketChannelTests Remove QueryParseContext from parsing QueryBuilders (#25448) Promote replica on the highest version node (#25277) test: added not null assertion ...
This pull request removes shared attributes from the Versions.asciidoc file and instead adds a reference to the list of shared attributes in the elastic/docs/shared repository.