{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/15945","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/15945/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/15945/comments","events_url":"https://api.github.com/repos/apache/spark/issues/15945/events","html_url":"https://github.com/apache/spark/pull/15945","id":190565171,"number":15945,"title":"[SPARK-12978][SQL] Merge unnecessary partial aggregates","user":{"login":"maropu","id":692303,"avatar_url":"https://avatars.githubusercontent.com/u/692303?v=3","gravatar_id":"","url":"https://api.github.com/users/maropu","html_url":"https://github.com/maropu","followers_url":"https://api.github.com/users/maropu/followers","following_url":"https://api.github.com/users/maropu/following{/other_user}","gists_url":"https://api.github.com/users/maropu/gists{/gist_id}","starred_url":"https://api.github.com/users/maropu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maropu/subscriptions","organizations_url":"https://api.github.com/users/maropu/orgs","repos_url":"https://api.github.com/users/maropu/repos","events_url":"https://api.github.com/users/maropu/events{/privacy}","received_events_url":"https://api.github.com/users/maropu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-11-20T15:43:40Z","updated_at":"2016-11-20T17:50:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/15945","html_url":"https://github.com/apache/spark/pull/15945","diff_url":"https://github.com/apache/spark/pull/15945.diff","patch_url":"https://github.com/apache/spark/pull/15945.patch"},"body":"## What changes were proposed in this pull request?\r\nThis pr is to merge unnecessary partial aggregates if the inputs of aggregates satisfy the distribution requirement of these partial aggregates. This pr is rework based on the @cloud-fan 's suggestion in #14909.\r\n\r\n## How was this patch tested?\r\nAdd tests in `PlannerSuite` to check if these partial aggregates are removed by catalyst.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/261793187","html_url":"https://github.com/apache/spark/pull/15945#issuecomment-261793187","issue_url":"https://api.github.com/repos/apache/spark/issues/15945","id":261793187,"user":{"login":"SparkQA","id":8108735,"avatar_url":"https://avatars.githubusercontent.com/u/8108735?v=3","gravatar_id":"","url":"https://api.github.com/users/SparkQA","html_url":"https://github.com/SparkQA","followers_url":"https://api.github.com/users/SparkQA/followers","following_url":"https://api.github.com/users/SparkQA/following{/other_user}","gists_url":"https://api.github.com/users/SparkQA/gists{/gist_id}","starred_url":"https://api.github.com/users/SparkQA/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SparkQA/subscriptions","organizations_url":"https://api.github.com/users/SparkQA/orgs","repos_url":"https://api.github.com/users/SparkQA/repos","events_url":"https://api.github.com/users/SparkQA/events{/privacy}","received_events_url":"https://api.github.com/users/SparkQA/received_events","type":"User","site_admin":false},"created_at":"2016-11-20T17:50:16Z","updated_at":"2016-11-20T17:50:16Z","body":"**[Test build #68901 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68901/consoleFull)** for PR 15945 at commit [`2633ced`](https://github.com/apache/spark/commit/2633ceddf5d10472aea0e33ca5d5f0022879dbc5).\n * This patch passes all tests.\n * This patch merges cleanly.\n * This patch adds the following public classes _(experimental)_:\n  * `class QueryExecution(val sparkSession: SparkSession, val logical: LogicalPlan)`\n  * `abstract class AggregateExec extends UnaryExecNode `"}}
{"actor":{"display_login":"SparkQA"}}