[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-17 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 7: (1 comment)

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-16 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 8: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4673 To

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-16 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. IMPALA-2789: More compact mem layout with null bits at the end. There are two

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-16 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 7: (1 comment)

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-16 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 7: (1 comment)

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-16 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 8: Code-Review+2 Carry +2 from Marcel -- To view, visit

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-16 Thread Alex Behm (Code Review)
Hello Marcel Kornacker, Matthew Jacobs, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4673 to look at the new patch set (#8). Change subject: IMPALA-2789: More compact mem layout with null bits at the end.

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-16 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 7: (1 comment)

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-12 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 7: Code-Review+1 Thanks! -- To view, visit http://gerrit.cloudera.org:8080/4673

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-12 Thread Alex Behm (Code Review)
Hello Matthew Jacobs, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4673 to look at the new patch set (#7). Change subject: IMPALA-2789: More compact mem layout with null bits at the end.

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-12 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/4673/6//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-12 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 6: Code-Review+1 (2 comments) Thanks! LGTM, just a few comments for the commit

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 6: Code-Review+1 Carry Tim's +1 -- To view, visit

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 5: (4 comments)

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Alex Behm (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4673 to look at the new patch set (#6). Change subject: IMPALA-2789: More compact mem layout with null bits at the end.

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 5: Code-Review+1 (4 comments)

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#5). Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. IMPALA-2789: More compact mem layout with null bits at the end. The main motivation of this change

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 4: (6 comments) http://gerrit.cloudera.org:8080/#/c/4673/2/be/src/exec/hdfs-scanner.cc

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 4: (6 comments) Looking pretty good

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#4). Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. IMPALA-2789: More compact mem layout with null bits at the end. The main motivation of this change

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#3). Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. IMPALA-2789: More compact mem layout with null bits at the end. The main motivation of this change

[Impala-ASF-CR] IMPALA-2789: More compact mem layout with null bits at the end.

2016-10-11 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2789: More compact mem layout with null bits at the end. .. Patch Set 2: (8 comments) http://gerrit.cloudera.org:8080/#/c/4673/2/be/src/exec/hdfs-scanner.cc