[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-30 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r206295564 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,25 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-30 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r206295347 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/ClientHashAggregateIT.java --- @@ -0,0 +1,191 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-25 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r205297065 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,25 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-25 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r205296815 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,25 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-25 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r205296403 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/ClientHashAggregateIT.java --- @@ -0,0 +1,191 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-25 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r205296128 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -183,13 +198,15 @@ public ExplainPlan getExplainPlan()

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-25 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r205232207 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -183,13 +198,15 @@ public ExplainPlan

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-25 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r205229353 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/ClientHashAggregateIT.java --- @@ -0,0 +1,191 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-25 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r205231380 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,25 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-25 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r205230186 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,25 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-25 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r205206944 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,24 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-23 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204495020 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,24 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204197328 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,24 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204197232 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204196968 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + *

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r20419 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204192760 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204192752 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204192739 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204167291 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + *

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204161104 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + *

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204176238 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + *

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-20 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r204168576 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,189 @@ +/* + *

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-19 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r203926206 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-19 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r203926179 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-19 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r203926229 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,24 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-19 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r203867850 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,173 @@ +/* + *

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-19 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r203882988 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashSizeException.java --- @@ -0,0 +1,33 @@ +/* + * Licensed to the

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-19 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r203868079 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,173 @@ +/* + *

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-19 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r203879864 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/execute/ClientAggregatePlan.java --- @@ -135,17 +142,24 @@ public ResultIterator

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-18 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r203467413 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-17 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r203099292 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-16 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r202842452 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + *

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-16 Thread solzy
Github user solzy commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r202593990 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + * Licensed to

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-12 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r202112790 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-12 Thread solzy
Github user solzy commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r202024941 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + * Licensed to

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-10 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r201562225 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-10 Thread geraldss
Github user geraldss commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r201562154 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-10 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r201468535 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-10 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/308#discussion_r201465583 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ClientHashAggregatingResultIterator.java --- @@ -0,0 +1,155 @@ +/* + * Licensed

[GitHub] phoenix pull request #308: Client-side hash aggregation

2018-07-02 Thread geraldss
GitHub user geraldss opened a pull request: https://github.com/apache/phoenix/pull/308 Client-side hash aggregation Client-side hash aggregation for use with sort-merge join. Implements https://issues.apache.org/jira/browse/PHOENIX-4751 You can merge this pull request