Skip to content

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 lcawl added >docs General docs changes v5.5.0 v5.6.0 v6.0.0 labels Jun 29, 2017
@lcawl lcawl requested review from clintongormley and debadair June 29, 2017 17:43
@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 lcawl force-pushed the lcawley-sharedattrs branch from 20f4f2e to 0614e25 Compare June 29, 2017 19:27
@lcawl lcawl force-pushed the lcawley-sharedattrs branch from 0614e25 to 95926c0 Compare June 29, 2017 20:00
@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