[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 5: Verified+1 -- To view, visit http:/

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. IMPALA-6383: free memory after skipping pa

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-24 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 5: Code-Review+2 -- To view, visit http://gerr

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 5: Build started: https://jenkins.impal

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-24 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 4: Code-Review+2 Checked with Alex that he has no

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-23 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 4: I'm happy to +2 this one. Alex, do you have an

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 4: Anyone want to do a +2 review? -- To view,

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 4: Code-Review+1 Carry +1 -- To view, visit h

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Tim Armstrong (Code Review)
Hello Lars Volker, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9059 to look at the new patch set (#3). Change subject: IMPALA-6383: free memory after skipping parquet row groups .

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has restored this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Restored -- To view, visit http://gerrit.cloudera.org:8080/9059 To un

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Tim Armstrong (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9059 to look at the new patch set (#2). Change subject: IMPALA-6383: free memory after skipping parquet row groups .. IMPALA

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has abandoned this change. ( http://gerrit.cloudera.org:8080/9059 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/9059 To

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9059 Change subject: IMPALA-6383: free memory after skipping parquet row groups .. IMPALA-6383: free memory after skipping parquet row groups

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. IMPALA-6383: free memory after skipping pa

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 6: Verified+1 -- To view, visit http:/

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 6: Code-Review+2 carry -- To view, visit http

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 6: Build started: https://jenkins.impal

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.c

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9002 to look at the new patch set (#5). Change subject: IMPALA-6383: free memory after skipping parquet row groups .

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9002 to look at the new patch set (#3). Change subject: IMPALA-6383: free memory after skipping parquet row groups

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 2: Code-Review+1 (2 comments) http://gerrit.clou

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:808

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-11 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9002 to look at the new patch set (#2). Change subject: IMPALA-6383: free memory after skipping parquet row groups

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-11 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9002 ) Change subject: IMPALA-6383: free memory after skipping parquet row groups .. Patch Set 1: (3 comments) Is there a way to test this? ht

[Impala-ASF-CR] IMPALA-6383: free memory after skipping parquet row groups

2018-01-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9002 Change subject: IMPALA-6383: free memory after skipping parquet row groups .. IMPALA-6383: free memory after skipping parquet row groups