{"action":"created","issue":{"url":"https://api.github.com/repos/ing-bank/scruid/issues/31","repository_url":"https://api.github.com/repos/ing-bank/scruid","labels_url":"https://api.github.com/repos/ing-bank/scruid/issues/31/labels{/name}","comments_url":"https://api.github.com/repos/ing-bank/scruid/issues/31/comments","events_url":"https://api.github.com/repos/ing-bank/scruid/issues/31/events","html_url":"https://github.com/ing-bank/scruid/pull/31","id":353301589,"node_id":"MDExOlB1bGxSZXF1ZXN0MjEwMzg3MjE2","number":31,"title":"Post-aggregations","user":{"login":"vkorchik","id":31250785,"node_id":"MDQ6VXNlcjMxMjUwNzg1","avatar_url":"https://avatars1.githubusercontent.com/u/31250785?v=4","gravatar_id":"","url":"https://api.github.com/users/vkorchik","html_url":"https://github.com/vkorchik","followers_url":"https://api.github.com/users/vkorchik/followers","following_url":"https://api.github.com/users/vkorchik/following{/other_user}","gists_url":"https://api.github.com/users/vkorchik/gists{/gist_id}","starred_url":"https://api.github.com/users/vkorchik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vkorchik/subscriptions","organizations_url":"https://api.github.com/users/vkorchik/orgs","repos_url":"https://api.github.com/users/vkorchik/repos","events_url":"https://api.github.com/users/vkorchik/events{/privacy}","received_events_url":"https://api.github.com/users/vkorchik/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2018-08-23T09:35:44Z","updated_at":"2018-08-29T07:51:47Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/ing-bank/scruid/pulls/31","html_url":"https://github.com/ing-bank/scruid/pull/31","diff_url":"https://github.com/ing-bank/scruid/pull/31.diff","patch_url":"https://github.com/ing-bank/scruid/pull/31.patch"},"body":"1) Added all post-aggregations from Druid documetation except just one: HyperUnique Cardinality. Simply because we dont have aggregations of the same type, and not to confuse users. Anyway, it can be added later without any troubles.\r\n2) Arithmetic functions for `ArithmeticPostAggregation` are coded as a separate family of case classes. It gives users only the defined set of functions to choose from, and not put \"blah-blah\" into `function` field.\r\n3) Added implicit conversions for arithmetic functions, as it have been already done for Filters and Havings. Precedence of operators of course is preserved, but remember that `quotient` has the lowest precedence, when using implicit conversions. "},"comment":{"url":"https://api.github.com/repos/ing-bank/scruid/issues/comments/416859688","html_url":"https://github.com/ing-bank/scruid/pull/31#issuecomment-416859688","issue_url":"https://api.github.com/repos/ing-bank/scruid/issues/31","id":416859688,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNjg1OTY4OA==","user":{"login":"Fokko","id":1134248,"node_id":"MDQ6VXNlcjExMzQyNDg=","avatar_url":"https://avatars0.githubusercontent.com/u/1134248?v=4","gravatar_id":"","url":"https://api.github.com/users/Fokko","html_url":"https://github.com/Fokko","followers_url":"https://api.github.com/users/Fokko/followers","following_url":"https://api.github.com/users/Fokko/following{/other_user}","gists_url":"https://api.github.com/users/Fokko/gists{/gist_id}","starred_url":"https://api.github.com/users/Fokko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Fokko/subscriptions","organizations_url":"https://api.github.com/users/Fokko/orgs","repos_url":"https://api.github.com/users/Fokko/repos","events_url":"https://api.github.com/users/Fokko/events{/privacy}","received_events_url":"https://api.github.com/users/Fokko/received_events","type":"User","site_admin":false},"created_at":"2018-08-29T07:51:47Z","updated_at":"2018-08-29T07:51:47Z","author_association":"MEMBER","body":"@krisgeus Awesome, I've retriggered the build."}}
{"actor":{"display_login":"Fokko"}}