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] Update shared attributes for Elasticsearch #25479

Merged
merged 3 commits into from
Jun 29, 2017

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Jun 29, 2017

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.

@lcawl
Copy link
Contributor Author

lcawl commented Jun 29, 2017

@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.

Copy link
Contributor

@debadair debadair left a 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.

@debadair
Copy link
Contributor

BTW, I'd be inclined to pull the attributes for the definitive guide and painless and plugins docs into the shared file, as well.

@lcawl
Copy link
Contributor Author

lcawl commented Jun 29, 2017

@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.

@debadair
Copy link
Contributor

LGTM

@lcawl lcawl merged commit 57c7520 into elastic:master Jun 29, 2017
@lcawl lcawl deleted the lcawley-sharedattrs branch June 29, 2017 20:49
lcawl added a commit that referenced this pull request Jun 29, 2017
* [DOCS] Update shared attributes for Elasticsearch

* [DOCS] Moved shared attributes to Versions.asciidoc

* [DOCS] More more book URLs to shared attributes
lcawl added a commit that referenced this pull request Jun 29, 2017
* [DOCS] Update shared attributes for Elasticsearch

* [DOCS] Moved shared attributes to Versions.asciidoc

* [DOCS] More more book URLs to shared attributes
@lcawl
Copy link
Contributor Author

lcawl commented Jun 29, 2017

Thanks! I've retrofitted to 5.x and 5.5

jasontedor added a commit that referenced this pull request Jun 30, 2017
* 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
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants