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.

Sign up for GitHub

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

Jump to bottom

Remove index.ttl.disable_purge #27582

Closed
clintongormley opened this issue Nov 29, 2017 · 7 comments
Closed

Remove index.ttl.disable_purge #27582

clintongormley opened this issue Nov 29, 2017 · 7 comments
Labels
:Core/Infra/Core Core issues without another label good first issue low hanging fruit help wanted adoptme >non-issue

Comments

@clintongormley
Copy link
Contributor

TTL is no longer supported and the index.ttl.disable_purge setting can be removed.
@PnPie PnPie mentioned this issue Nov 29, 2017
Closed
@PnPie
Copy link
Contributor

PnPie commented Nov 29, 2017

I make one PR for this one :)

luizgpsantos added a commit to luizgpsantos/elasticsearch that referenced this issue Dec 8, 2017
@luizgpsantos
Removes index.ttl.disable_purge as TTL is no longer supported. Relates …
7ae6802 
…elastic#27582
@luizgpsantos luizgpsantos mentioned this issue Dec 8, 2017
Closed
@videetssinghai
Copy link

I would like to make a PR for this one if this issue is still open

@videetssinghai
Copy link

Ok so I found "index.ttl.disable_purge" in 3 files - IndexScopedSettings, IndexSettings and IndexSettingsTest. How do I remove it ? ( straight away remove its functions and tests without affecting the other code ?)

@videetssinghai
Copy link

@clintongormley

@javanna
Copy link
Member

javanna commented Mar 1, 2018

hi @videetssinghai there is already a PR waiting for review that fixes this issue, see #27589 .

@colings86 colings86 added the :Core/Infra/Core Core issues without another label label Apr 24, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@DaveCTurner
Copy link
Contributor

Sorry, I didn't see this was open. It was deprecated in #29526 and closed in #29527, so this can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Core/Infra/Core Core issues without another label good first issue low hanging fruit help wanted adoptme >non-issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@clintongormley @colings86 @javanna @DaveCTurner @PnPie @videetssinghai @elasticmachine