[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 17: Verified+1 -- To view, visit http://ge

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. IMPALA-5844: use a MemPool for expr result all

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 16: Verified-1 Build failed: https://jenkin

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 17: Build started: https://jenkins.impala.i

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 17: Code-Review+2 Hit a clang-tidy error: 19:29:2

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-05 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Dan Hecht, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#17). Change subject: IMPALA-5844: use a MemPool for expr result allocations ..

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 16: Build started: https://jenkins.impala.i

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-05 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#16). Change subject: IMPALA-5844: use a MemPool for expr result allocations .

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 16: Code-Review+2 rebase -- To view, visit http:/

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 15: Code-Review+2 PS13 was the rebase (required to

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-04 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#14). Change subject: IMPALA-5844: use a MemPool for expr result allocations .

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-04 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#15). Change subject: IMPALA-5844: use a MemPool for expr result allocations .

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 12: (6 comments) Rebase, then addressed comments.

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-04 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#13). Change subject: IMPALA-5844: use a MemPool for expr result allocations .

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-04 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 12: Code-Review+2 (5 comments) http://gerrit.cloudera.

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-03 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#12). Change subject: IMPALA-5844: use a MemPool for expr result allocations .

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 11: (17 comments) http://gerrit.cloudera.org:8080/

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-03 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 11: (4 comments) Remaining comments. http://gerrit.cl

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-10-03 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 11: (12 comments) This is looking very good. Todo for

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-09-28 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-09-28 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/80

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-09-28 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 9: (11 comments) http://gerrit.cloudera.org:8080/#

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-09-28 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#10). Change subject: IMPALA-5844: use a MemPool for expr result allocations .. IMPALA-58

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-09-28 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 9: (11 comments) http://gerrit.cloudera.org:8080/#/c/8

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-09-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 9: Rebased -- To view, visit http://gerrit.cloude

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-09-25 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#9). Change subject: IMPALA-5844: use a MemPool for expr result allocations .. IMPALA-584

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-09-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr result allocations .. Patch Set 7: (5 comments) I think I addressed all the commen

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr result allocations

2017-09-25 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#8). Change subject: IMPALA-5844: use a MemPool for expr result allocations .. IMPALA-584