{"action":"created","issue":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/13672","labels_url":"https://api.github.com/repos/elastic/elasticsearch/issues/13672/labels{/name}","comments_url":"https://api.github.com/repos/elastic/elasticsearch/issues/13672/comments","events_url":"https://api.github.com/repos/elastic/elasticsearch/issues/13672/events","html_url":"https://github.com/elastic/elasticsearch/pull/13672","id":107474193,"number":13672,"title":"WIP: Switch geo validation to enum","user":{"login":"MaineC","id":70953,"avatar_url":"https://avatars.githubusercontent.com/u/70953?v=3","gravatar_id":"","url":"https://api.github.com/users/MaineC","html_url":"https://github.com/MaineC","followers_url":"https://api.github.com/users/MaineC/followers","following_url":"https://api.github.com/users/MaineC/following{/other_user}","gists_url":"https://api.github.com/users/MaineC/gists{/gist_id}","starred_url":"https://api.github.com/users/MaineC/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MaineC/subscriptions","organizations_url":"https://api.github.com/users/MaineC/orgs","repos_url":"https://api.github.com/users/MaineC/repos","events_url":"https://api.github.com/users/MaineC/events{/privacy}","received_events_url":"https://api.github.com/users/MaineC/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/:Query%20Refactoring","name":":Query Refactoring","color":"009800"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/v3.0.0","name":"v3.0.0","color":"ededed"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/WIP","name":"WIP","color":"fef2c0"}],"state":"open","locked":false,"assignee":{"login":"MaineC","id":70953,"avatar_url":"https://avatars.githubusercontent.com/u/70953?v=3","gravatar_id":"","url":"https://api.github.com/users/MaineC","html_url":"https://github.com/MaineC","followers_url":"https://api.github.com/users/MaineC/followers","following_url":"https://api.github.com/users/MaineC/following{/other_user}","gists_url":"https://api.github.com/users/MaineC/gists{/gist_id}","starred_url":"https://api.github.com/users/MaineC/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MaineC/subscriptions","organizations_url":"https://api.github.com/users/MaineC/orgs","repos_url":"https://api.github.com/users/MaineC/repos","events_url":"https://api.github.com/users/MaineC/events{/privacy}","received_events_url":"https://api.github.com/users/MaineC/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-09-21T09:23:10Z","updated_at":"2015-09-21T11:48:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/elastic/elasticsearch/pulls/13672","html_url":"https://github.com/elastic/elasticsearch/pull/13672","diff_url":"https://github.com/elastic/elasticsearch/pull/13672.diff","patch_url":"https://github.com/elastic/elasticsearch/pull/13672.patch"},"body":"Relates to #13608:\r\n\r\nThis switches handling coerce and ignore_malformed parameters to an enum to better reflect their relationship.\r\n\r\n@cbuescher / @nknize / @colings86 can either of you have a look please whether those changes make sense and are in line with the issue above before I move on to the other geo queries."},"comment":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/comments/141950131","html_url":"https://github.com/elastic/elasticsearch/pull/13672#issuecomment-141950131","issue_url":"https://api.github.com/repos/elastic/elasticsearch/issues/13672","id":141950131,"user":{"login":"cbuescher","id":10398885,"avatar_url":"https://avatars.githubusercontent.com/u/10398885?v=3","gravatar_id":"","url":"https://api.github.com/users/cbuescher","html_url":"https://github.com/cbuescher","followers_url":"https://api.github.com/users/cbuescher/followers","following_url":"https://api.github.com/users/cbuescher/following{/other_user}","gists_url":"https://api.github.com/users/cbuescher/gists{/gist_id}","starred_url":"https://api.github.com/users/cbuescher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cbuescher/subscriptions","organizations_url":"https://api.github.com/users/cbuescher/orgs","repos_url":"https://api.github.com/users/cbuescher/repos","events_url":"https://api.github.com/users/cbuescher/events{/privacy}","received_events_url":"https://api.github.com/users/cbuescher/received_events","type":"User","site_admin":false},"created_at":"2015-09-21T11:48:25Z","updated_at":"2015-09-21T11:48:25Z","body":"@MaineC Looks good, just left a few minor suggestions. Maybe @nknize can comment on whether the enum is clear enough from user perspective to remove the two independent setters from the java api."}}