[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.cloud

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. IMPALA-7296: bytes limit for row batch queue https://g

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cl

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 7: Build started: https://jenkins.impala.io/job/gerr

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-30 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-26 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/10977/5/

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/10977/5/be

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 6: Build Successful https://jenkins.impala.io/job/g

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 6: Build Started https://jenkins.impala.io/job/gerri

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-25 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10977 to look at the new patch set (#6). Change subject: IMPALA-7296: bytes limit for row batch queue .. IMPAL

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 5: (9 comments) http://gerrit.cloudera.org:8080/#/c/10977/5

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-24 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/10977/5/be/

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/10977/5/b

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-24 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 5: (5 comments) http://gerrit.cloudera.org:8080/#/c/10977/5/be

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 5: Mainly to avoid potentially making things worse for narro

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-20 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 5: Hi Tim, will it be simpler in general to just use max_bytes

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. IMPALA-7296: bytes limit for row batch queue https://goo.gl/N9LgQt summa

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-18 Thread Tim Armstrong (Code Review)
Hello Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10977 to look at the new patch set (#4). Change subject: IMPALA-7296: bytes limit for row batch queue .. IMPALA-7296: bytes

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. Patch Set 3: kwho, would be nice for you to have a look when you have

[Impala-ASF-CR] IMPALA-7296: bytes limit for row batch queue

2018-07-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10977 ) Change subject: IMPALA-7296: bytes limit for row batch queue .. IMPALA-7296: bytes limit for row batch queue https://goo.gl/N9LgQt summa