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

_ttl: enabled: false has no effect #7293

Closed
brwe opened this issue Aug 15, 2014 · 2 comments
Closed

_ttl: enabled: false has no effect #7293

brwe opened this issue Aug 15, 2014 · 2 comments
Assignees

Comments

@brwe
Copy link
Contributor

brwe commented Aug 15, 2014

(Updated - my first examples were wrong)

In the following example the document will be deleted:

DELETE testidx
PUT testidx
PUT testidx/doc/_mapping
{
  "_ttl":{
    "enabled": "true"
  }
}

GET testidx/_mapping


PUT testidx/doc/_mapping
{
  "_ttl":{
    "enabled": "false"
  }
}
# _ttl is enabled anyway
GET testidx/_mapping
POST testidx/doc/1
{
  "text":"foo",
  "_ttl": "10ms"
}

#document will be deleted after a while
GET testidx/doc/1
@brwe brwe changed the title _ttl: Inconsistency when enabling _ttl in mapping _ttl: enabled: no has no effect Aug 15, 2014
@brwe brwe self-assigned this Aug 15, 2014
@brwe brwe changed the title _ttl: enabled: no has no effect _ttl: enabled: false has no effect Aug 15, 2014
@clintongormley
Copy link

@brwe Isn't this a duplicate of #777?

@brwe
Copy link
Contributor Author

brwe commented Aug 18, 2014

When I opened the issue it was not clear to me if one should be allowed to disable _ttl dynamically. After reading more code I think it should not be possible so yes, a duplicate of #777.

@brwe brwe closed this as completed Aug 18, 2014
brwe added a commit to brwe/elasticsearch that referenced this issue Aug 18, 2014
_ttl could never be disabled once it was enabled.
But when trying to, no conflict was reported.

relates to elastic#777 and elastic#7293
brwe added a commit that referenced this issue Aug 21, 2014
_ttl could never be disabled once it was enabled.
But when trying to, no conflict was reported.

relates to #777 and #7293

closes #7316
brwe added a commit that referenced this issue Aug 21, 2014
_ttl could never be disabled once it was enabled.
But when trying to, no conflict was reported.

relates to #777 and #7293

closes #7316
brwe added a commit that referenced this issue Sep 8, 2014
_ttl could never be disabled once it was enabled.
But when trying to, no conflict was reported.

relates to #777 and #7293

closes #7316
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants