{"action":"created","issue":{"url":"https://api.github.com/repos/elasticsearch/elasticsearch/issues/9521","labels_url":"https://api.github.com/repos/elasticsearch/elasticsearch/issues/9521/labels{/name}","comments_url":"https://api.github.com/repos/elasticsearch/elasticsearch/issues/9521/comments","events_url":"https://api.github.com/repos/elasticsearch/elasticsearch/issues/9521/events","html_url":"https://github.com/elasticsearch/elasticsearch/pull/9521","id":56177188,"number":9521,"title":"parent/child: Added `strict` option that controls how to deal with no or incorrectly configured _parent field mapping.","user":{"login":"martijnvg","id":580421,"avatar_url":"https://avatars.githubusercontent.com/u/580421?v=3","gravatar_id":"","url":"https://api.github.com/users/martijnvg","html_url":"https://github.com/martijnvg","followers_url":"https://api.github.com/users/martijnvg/followers","following_url":"https://api.github.com/users/martijnvg/following{/other_user}","gists_url":"https://api.github.com/users/martijnvg/gists{/gist_id}","starred_url":"https://api.github.com/users/martijnvg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martijnvg/subscriptions","organizations_url":"https://api.github.com/users/martijnvg/orgs","repos_url":"https://api.github.com/users/martijnvg/repos","events_url":"https://api.github.com/users/martijnvg/events{/privacy}","received_events_url":"https://api.github.com/users/martijnvg/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/elasticsearch/elasticsearch/labels/enhancement","name":"enhancement","color":"4a4ea8"},{"url":"https://api.github.com/repos/elasticsearch/elasticsearch/labels/review","name":"review","color":"0052cc"},{"url":"https://api.github.com/repos/elasticsearch/elasticsearch/labels/v1.5.0","name":"v1.5.0","color":"dddddd"},{"url":"https://api.github.com/repos/elasticsearch/elasticsearch/labels/v2.0.0","name":"v2.0.0","color":"dddddd"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2015-02-01T20:39:18Z","updated_at":"2015-02-11T01:12:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/elasticsearch/elasticsearch/pulls/9521","html_url":"https://github.com/elasticsearch/elasticsearch/pull/9521","diff_url":"https://github.com/elasticsearch/elasticsearch/pull/9521.diff","patch_url":"https://github.com/elasticsearch/elasticsearch/pull/9521.patch"},"body":"Added `strict` boolean option to `has_child` and `has_parent` queries. If set to true (default) no or an incorrect `_parent` field results in a parse error, if set to false then the query will not fail and yield no results.\r\n\r\nThis can be useful when querying across multiple indices where not all indices have a `_parent` field defined.\r\n\r\nPR fixes #9461 too. The query parsing failed sometimes with a unexpected NPE, because the `_parent` field is pointing to a non existing type. The consistent strict / non strict mode makes sure the NPE can't occur any more.\r\n\r\nThis PR also includes a commit that removes the old and used `_cache` and `_cache_key` options."},"comment":{"url":"https://api.github.com/repos/elasticsearch/elasticsearch/issues/comments/73818549","html_url":"https://github.com/elasticsearch/elasticsearch/pull/9521#issuecomment-73818549","issue_url":"https://api.github.com/repos/elasticsearch/elasticsearch/issues/9521","id":73818549,"user":{"login":"rjernst","id":289412,"avatar_url":"https://avatars.githubusercontent.com/u/289412?v=3","gravatar_id":"","url":"https://api.github.com/users/rjernst","html_url":"https://github.com/rjernst","followers_url":"https://api.github.com/users/rjernst/followers","following_url":"https://api.github.com/users/rjernst/following{/other_user}","gists_url":"https://api.github.com/users/rjernst/gists{/gist_id}","starred_url":"https://api.github.com/users/rjernst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rjernst/subscriptions","organizations_url":"https://api.github.com/users/rjernst/orgs","repos_url":"https://api.github.com/users/rjernst/repos","events_url":"https://api.github.com/users/rjernst/events{/privacy}","received_events_url":"https://api.github.com/users/rjernst/received_events","type":"User","site_admin":false},"created_at":"2015-02-11T01:12:41Z","updated_at":"2015-02-11T01:12:41Z","body":"> Maybe as part of the work we should change the index.query.parse.allow_unmapped_fields option into index.query.parse.unmapped_fields with the same values as this new request flag?\r\n\r\n+1.  I would also rather use the values `ignore`, `warn` and `error` then (and preferably default to `error` although I know that is probably contentious)."}}