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

Explicitly recommend to forceMerge before freezing #36376

Merged
merged 5 commits into from
Dec 7, 2018

Conversation

s1monw
Copy link
Contributor

@s1monw s1monw commented Dec 7, 2018

given the benchmark results on #34352 it's important to recommend
users to _force_merge their indices to a single segment before freezing.

given the benchmark results on elastic#34352 it's important to recommend
users to `_force_merge` their indice to a single segment before freezing.
@s1monw s1monw added >docs General docs changes v7.0.0 v6.6.0 labels Dec 7, 2018
Copy link
Contributor

@clintongormley clintongormley left a comment

Choose a reason for hiding this comment

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

LGTM after changes are accepted

docs/reference/frozen-indices.asciidoc Outdated Show resolved Hide resolved
docs/reference/frozen-indices.asciidoc Outdated Show resolved Hide resolved
docs/reference/frozen-indices.asciidoc Outdated Show resolved Hide resolved
clintongormley and others added 4 commits December 7, 2018 18:12
Co-Authored-By: s1monw <simonw@apache.org>
Co-Authored-By: s1monw <simonw@apache.org>
Co-Authored-By: s1monw <simonw@apache.org>
Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

LGTM

@s1monw s1monw merged commit 2ba023b into elastic:master Dec 7, 2018
@s1monw s1monw deleted the recommend_force_merge branch December 7, 2018 22:39
s1monw added a commit that referenced this pull request Dec 7, 2018
given the benchmark results on #34352 it's important to recommend
users to `_force_merge` their indices to a single segment before freezing.
jasontedor added a commit to liketic/elasticsearch that referenced this pull request Dec 9, 2018
* elastic/6.x: (37 commits)
  [HLRC] Added support for Follow Stats API (elastic#36253)
  Exposed engine must have all ops below gcp during rollback (elastic#36159)
  TEST: Always enable soft-deletes in ShardChangesTests
  Use delCount of SegmentInfos to calculate numDocs (elastic#36323)
  Add soft-deletes upgrade tests (elastic#36286)
  Remove LocalCheckpointTracker#resetCheckpoint (elastic#34667)
  Option to use endpoints starting with _security (elastic#36379)
  [CCR] Restructured QA modules (elastic#36404)
  RestClient: on retry timeout add root exception (elastic#25576)
  [HLRC] Add support for put privileges API (elastic#35679)
  HLRC: Add rollup search (elastic#36334)
  Explicitly recommend to forceMerge before freezing (elastic#36376)
  Rename internal repository actions to be internal (elastic#36377)
  Core: Remove parseDefaulting from DateFormatter (elastic#36386)
  [ML] Prevent stack overflow while copying ML jobs and datafeeds (elastic#36370)
  Docs: Fix Jackson reference (elastic#36366)
  [ILM] Fix issue where index may not yet be in 'hot' phase (elastic#35716)
  Undeprecate /_watcher endpoints (elastic#36269)
  Docs: Fix typo in bool query (elastic#36350)
  HLRC: Add delete template API (elastic#36320)
  ...
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