-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[indexPattern/field] remove "bucketable" property of fields #5806
Conversation
We currently setup a bucketable property on every field in order to detect if the field can be used in a bucket aggregation. When it was discovered that our condition was incorrect we decided that we wanted to lean on Elasticsearch's new strctured error messages rather than trying to maintain a secondary implementation of the validation in Kibana.
not sure to understand why but with latest released kibana and ES:
this pull request saved my day (but the root problem is still here: why my "non string" fields are considered as non indexed ?) an idea ? or I open a new issue (about the root cause) ? |
corresponding simple mapping template: {
"template": "nginx-*",
"mappings": {
"nginx": {
"properties": {
"@timestamp": {"type": "date"},
"source_host": {"type": "string", "index": "not_analyzed"},
"source_path": {"type": "string", "index": "not_analyzed"},
"module": {"type": "string", "index": "not_analyzed"},
"user_id": {"type": "string", "index": "not_analyzed"},
"user_agent": {"type": "string", "index": "not_analyzed"},
"uri": {"type": "string"},
"uri_path": {"type": "string", "index": "not_analyzed"},
"status": {"type": "integer"},
"root_id": {"type": "string", "index": "not_analyzed"},
"request_id": {"type": "string", "index": "not_analyzed"},
"mother_id": {"type": "string", "index": "not_analyzed"},
"request_length": {"type": "long"},
"reply_length": {"type": "long"},
"recursion": {"type": "string", "index": "not_analyzed"},
"processing_ms": {"type": "string", "index": "not_analyzed"},
"method": {"type": "string", "index": "not_analyzed"},
"from": {"type": "string", "index": "not_analyzed"},
"client_priority": {"type": "string", "index": "not_analyzed"},
"duration": {"type": "double"}
}
}
}
} |
I don't understand why this was merged? I didn't review it, and this definitely should not be in 4.5.0 anyway? |
@spalger Who reviewed this? |
@rashidkpc to be honest, I'm not sure who reviewed it, but I'm confident that I didn't just merge it... It didn't get backported to 4.5 so no worries there. |
Fixes #3335
Fixes #5914
We currently setup a
bucketable
property on every field in order to detect if the field can be used in a bucket aggregation. When it was discovered that our condition was incorrect we decided that we wanted to lean on Elasticsearch's new structured error messages rather than trying to maintain a secondary implementation of the validation in Kibana.