[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-28 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. IMPALA-7604: part 2: fixes for AggregationNode ca

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-28 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 10: Verified+1 -- To view, visit http

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 10: Code-Review+2 -- To view, visit h

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 10: Build started: https://jenkins.imp

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 9: Verified-1 Build failed: https://je

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 9: Build Successful https://jenkins.i

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 8: Build Successful https://jenkins.i

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 7: Build Successful https://jenkins.i

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 9: Build started: https://jenkins.impa

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 9: Code-Review+2 -- To view, visit http://ge

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 9: Changes weren't totally trivial, would be g

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Tim Armstrong (Code Review)
Hello Bharath Vissapragada, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14132 to look at the new patch set (#9). Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality ...

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Tim Armstrong (Code Review)
Hello Bharath Vissapragada, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14132 to look at the new patch set (#8). Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality ...

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 7: (6 comments) PS8 is a rebase onto the late

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Tim Armstrong (Code Review)
Hello Bharath Vissapragada, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14132 to look at the new patch set (#7). Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality ...

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 6: Code-Review+2 (1 comment) Lgtm http://ge

[Impala-ASF-CR] IMPALA-7604: part 2: fixes for AggregationNode cardinality

2019-08-27 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/14132 ) Change subject: IMPALA-7604: part 2: fixes for AggregationNode cardinality .. Patch Set 6: Code-Review+1 (4 comments) Patch an