Skip to content

Conversation

@lcawl
Copy link
Contributor

@lcawl lcawl commented Feb 14, 2019

Related to #37942 (comment)

This PR clarifies the options for stopping machine learning (or not) during an upgrade.

@lcawl lcawl added >docs General docs changes :ml Machine learning v8.0.0 labels Feb 14, 2019
@lcawl lcawl requested a review from droberts195 February 14, 2019 02:21
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

Copy link

@droberts195 droberts195 left a comment

Choose a reason for hiding this comment

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

LGTM

I think this should be backported to 6.7

@lcawl lcawl merged commit c3f0df3 into elastic:master Feb 15, 2019
@lcawl lcawl deleted the ml-upgrade branch February 15, 2019 16:23
lcawl added a commit to lcawl/elasticsearch that referenced this pull request Feb 15, 2019
lcawl added a commit to lcawl/elasticsearch that referenced this pull request Feb 15, 2019
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Feb 15, 2019
* master:
  Address some CCR REST test case flakiness (elastic#38975)
  Edits to text in Completion Suggester doc (elastic#38980)
  SQL: doc polishing
  [DOCS] Fixes broken formatting
  SQL: Polish the rest chapter (elastic#38971)
  Remove `nGram` and  `edgeNGram` token filter names (elastic#38911)
  Add an exception throw if waiting on transport port file fails (elastic#37574)
  Improve testcluster distribution artifact handling (elastic#38933)
  Advance max_seq_no before add operation to Lucene (elastic#38879)
  Reduce global checkpoint sync interval in disruption tests (elastic#38931)
  [test] disable packaging tests for suse boxes
  Relax testStressMaybeFlushOrRollTranslogGeneration (elastic#38918)
  [DOCS] Edits warning in put watch API (elastic#38582)
  Fix serialization bug in ShardFollowTask after cutting this class over to extend from ImmutableFollowParameters.
  [DOCS] Updates methods for upgrading machine learning (elastic#38876)
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.

4 participants