{"action":"created","issue":{"url":"https://api.github.com/repos/NLPchina/elasticsearch-sql/issues/66","labels_url":"https://api.github.com/repos/NLPchina/elasticsearch-sql/issues/66/labels{/name}","comments_url":"https://api.github.com/repos/NLPchina/elasticsearch-sql/issues/66/comments","events_url":"https://api.github.com/repos/NLPchina/elasticsearch-sql/issues/66/events","html_url":"https://github.com/NLPchina/elasticsearch-sql/pull/66","id":89592380,"number":66,"title":"Add aggregations to the query via GET or POST","user":{"login":"jheimbouch","id":4997688,"avatar_url":"https://avatars.githubusercontent.com/u/4997688?v=3","gravatar_id":"","url":"https://api.github.com/users/jheimbouch","html_url":"https://github.com/jheimbouch","followers_url":"https://api.github.com/users/jheimbouch/followers","following_url":"https://api.github.com/users/jheimbouch/following{/other_user}","gists_url":"https://api.github.com/users/jheimbouch/gists{/gist_id}","starred_url":"https://api.github.com/users/jheimbouch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jheimbouch/subscriptions","organizations_url":"https://api.github.com/users/jheimbouch/orgs","repos_url":"https://api.github.com/users/jheimbouch/repos","events_url":"https://api.github.com/users/jheimbouch/events{/privacy}","received_events_url":"https://api.github.com/users/jheimbouch/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-06-19T15:29:37Z","updated_at":"2015-06-20T19:11:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/NLPchina/elasticsearch-sql/pulls/66","html_url":"https://github.com/NLPchina/elasticsearch-sql/pull/66","diff_url":"https://github.com/NLPchina/elasticsearch-sql/pull/66.diff","patch_url":"https://github.com/NLPchina/elasticsearch-sql/pull/66.patch"},"body":"* Can specify aggregation fields through GET using \"&aggs=comma,delimited,field,names\"\r\n* Can specify aggregation fields through POST by adding \"<aggs>comma,delimited,field,names</aggs>\"\r\n* You can also add sub aggregates for an aggregate by using a colon - e.g. \"Firstname:age\""},"comment":{"url":"https://api.github.com/repos/NLPchina/elasticsearch-sql/issues/comments/113805152","html_url":"https://github.com/NLPchina/elasticsearch-sql/pull/66#issuecomment-113805152","issue_url":"https://api.github.com/repos/NLPchina/elasticsearch-sql/issues/66","id":113805152,"user":{"login":"jstortz","id":537809,"avatar_url":"https://avatars.githubusercontent.com/u/537809?v=3","gravatar_id":"","url":"https://api.github.com/users/jstortz","html_url":"https://github.com/jstortz","followers_url":"https://api.github.com/users/jstortz/followers","following_url":"https://api.github.com/users/jstortz/following{/other_user}","gists_url":"https://api.github.com/users/jstortz/gists{/gist_id}","starred_url":"https://api.github.com/users/jstortz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jstortz/subscriptions","organizations_url":"https://api.github.com/users/jstortz/orgs","repos_url":"https://api.github.com/users/jstortz/repos","events_url":"https://api.github.com/users/jstortz/events{/privacy}","received_events_url":"https://api.github.com/users/jstortz/received_events","type":"User","site_admin":false},"created_at":"2015-06-20T19:11:54Z","updated_at":"2015-06-20T19:11:54Z","body":"Omer, it is no trouble. We wanted to submit something so we could begin discussion with you and have something you can look at and comment on, just like you did.  Thank you for your feedback!\r\n\r\nI agree that a single aggregation, including sub-aggregration would work.  Because we need more than one we will have to push forward with our change.  You suggestions on the format are good.  We will talk about them Monday to see how our implementation can be changed to match that.  I like the look and feel of your suggestion.\r\n\r\nThanks!"}}