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

Forbid negative values for index.unassigned.node_left.delayed_timeout #26828

Merged

Conversation

DaveCTurner
Copy link
Contributor

Negative values for index.unassigned.node_left.delayed_timeout could be expected to mean "never" whereas in fact they mean "0". "Never" is not a useful value for this, so we should just reject negative values here.

Copy link
Contributor

@bleskes bleskes left a comment

Choose a reason for hiding this comment

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

Code looks great. We need a break change documentation. See docs/reference/migration/index.asciidoc

@DaveCTurner
Copy link
Contributor Author

@bleskes ok done.

@DaveCTurner DaveCTurner merged commit 8fe9a20 into elastic:master Sep 29, 2017
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Oct 1, 2017
* xdcr:
  Maybe die before trying to log cause
  Log cause when a write and flush fails
  Die if write listener fails due to fatal error
  RecoveryIT.testHistoryUUIDIsGenerated should reduce unassigned shards delay instead of ensure green.
  Replace group map settings with affix setting (elastic#26819)
  Fix references to vm.max_map_count in Docker docs
  Add more instructions about jar hell (elastic#26837)
  Forbid negative values for index.unassigned.node_left.delayed_timeout (elastic#26828)
  Nitpicking typos in comments (elastic#26831)
  MetaData Builder doesn't properly prevent an alias with the same name as an index (elastic#26804)
DaveCTurner added a commit that referenced this pull request Oct 9, 2017
…ed_timeout (#26832)

This was deprecated because it becomes forbidden in 7.x as per #26828.
@lcawl lcawl added :Search/Search Search-related issues that do not fall into other categories and removed :Allocation labels Feb 13, 2018
@clintongormley clintongormley added :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. and removed :Search/Search Search-related issues that do not fall into other categories labels Feb 13, 2018
@DaveCTurner DaveCTurner deleted the issue-26828-delayed_timeout-positive branch October 5, 2018 07:47
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Dec 10, 2018
…eout

This commit adds support to check for negative values for index setting
`index.unassigned.node_left.delayed_timeout` for the deprecation info API.

relates elastic#36024
relates elastic#26828
jakelandis added a commit that referenced this pull request Dec 13, 2018
…eout (#36454)

This commit adds support to check for negative values for index setting
`index.unassigned.node_left.delayed_timeout` for the deprecation info API.

relates #36024
relates #26828
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>breaking :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants