[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. IMPALA-6847: work around high memory estimates

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 14: Verified+1 -- To view, visit http://ge

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 14: Build started: https://jenkins.impala.i

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 14: Code-Review+2 -- To view, visit http://gerrit.

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 13: Code-Review+2 Looks good to me. Carrying over

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 13: Code-Review+2 Please wait for Bikram's sign off as

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#13). Change subject: IMPALA-6847: work around high memory estimates for AC

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 12: (7 comments) http://gerrit.cloudera.org:8080/#

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 12: (4 comments) http://gerrit.cloudera.org:8080/#/c/1

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 12: > > (3 comments) > > > I also really like th

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-17 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 12: > (3 comments) > I also really like the ideas in

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 12: (3 comments) +1 for the simplified version (p

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#12). Change subject: IMPALA-6847: work around high memory estimates for AC

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 11: (6 comments) Here's the new simplified version

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#11). Change subject: IMPALA-6847: work around high memory estimates for AC

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#10). Change subject: IMPALA-6847: work around high memory estimates for AC

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#9). Change subject: IMPALA-6847: work around high memory estimates for AC .

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 8: (6 comments) As we discussed offline, I think we'd

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#8). Change subject: IMPALA-6847: work around high memory estimates for AC .

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/10

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#7). Change subject: IMPALA-6847: work around high memory estimates for AC .

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/100

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#6). Change subject: IMPALA-6847: work around high memory estimates for AC ..

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#5). Change subject: IMPALA-6847: work around high memory estimates for AC ..

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-13 Thread Tim Armstrong (Code Review)
Hello Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#4). Change subject: IMPALA-6847: work around high memory estimates for AC ..

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-13 Thread Tim Armstrong (Code Review)
Hello Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#3). Change subject: IMPALA-6847: work around high memory estimates for AC ..