Skip to content

Add HLRC docs for Get Lifecycle Policy#35612

Merged
AthenaEryma merged 3 commits intoelastic:masterfrom
AthenaEryma:ilm/hlrc-get-policy-docs
Nov 17, 2018
Merged

Add HLRC docs for Get Lifecycle Policy#35612
AthenaEryma merged 3 commits intoelastic:masterfrom
AthenaEryma:ilm/hlrc-get-policy-docs

Conversation

@AthenaEryma
Copy link
Copy Markdown
Contributor

Adds docs for the Get Lifecycle Policy API to the HLRC documentation.

@AthenaEryma AthenaEryma added >docs General docs changes blocker :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. labels Nov 16, 2018
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra

Copy link
Copy Markdown
Contributor

@colings86 colings86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left some minor comments about adding asserts. I think this is important because without the asserts we aren't actually checking the code we are documenting is doing what we expect.

Copy link
Copy Markdown
Contributor

@talevy talevy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AthenaEryma AthenaEryma merged commit bb51cdb into elastic:master Nov 17, 2018
AthenaEryma added a commit that referenced this pull request Nov 17, 2018
Adds docs for the Get Lifecycle Policy API to the HLRC documentation.
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Nov 17, 2018
* master: (59 commits)
  SQL: Move internals from Joda to java.time (elastic#35649)
  Add HLRC docs for Get Lifecycle Policy (elastic#35612)
  Align RolloverStep's name with other step names (elastic#35655)
  Watcher: Use joda method to get local TZ (elastic#35608)
  Fix line length for org.elasticsearch.action.* files (elastic#35607)
  Remove use of AbstractComponent in server (elastic#35444)
  Deprecate types in count and msearch. (elastic#35421)
  Refactor an ambigious TermVectorsRequest constructor. (elastic#35614)
  [Scripting] Use Number as a return value for BucketAggregationScript (elastic#35653)
  Removes AbstractComponent from several classes (elastic#35566)
  [DOCS] Add beta warning to ILM pages. (elastic#35571)
  Deprecate types in validate query requests. (elastic#35575)
  Unmute BuildExamplePluginsIT
  Revert "AwaitsFix the RecoveryIT suite - see elastic#35597"
  Revert "[RCI] Check blocks while having index shard permit in TransportReplicationAction (elastic#35332)"
  Remove remaining line length violations for o.e.action.admin.cluster (elastic#35156)
  ML: Adjusing BWC version post backport to 6.6 (elastic#35605)
  [TEST] Replace fields in response with actual values
  Remove usages of CharSequence in Sets (elastic#35501)
  AwaitsFix the RecoveryIT suite - see elastic#35597
  ...
@AthenaEryma AthenaEryma deleted the ilm/hlrc-get-policy-docs branch December 7, 2018 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocker :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. >docs General docs changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants