[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-26 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 11: (1 comment)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 10: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. IMPALA-4236: Codegen CopyRows() for select nodes

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-24 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 10: Build started:

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-24 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 10: Code-Review+2 Rebased. carrying over Michael's +2

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-24 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 9: (2 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 9: Code-Review+2 (2 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-23 Thread Bikramjeet Vig (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8196 to look at the new patch set (#9). Change subject: IMPALA-4236: Codegen CopyRows() for select nodes ..

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-23 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 8: (2 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 8: (1 comment)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 8: (2 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 8: Code-Review+1 Will let Michael take another look.

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-16 Thread Bikramjeet Vig (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8196 to look at the new patch set (#8). Change subject: IMPALA-4236: Codegen CopyRows() for select nodes ..

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-16 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 7: (3 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 7: (4 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-10 Thread Bikramjeet Vig (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8196 to look at the new patch set (#7). Change subject: IMPALA-4236: Codegen CopyRows() for select nodes ..

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-10 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 6: (6 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-09 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 6: (1 comment)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-09 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 6: (1 comment)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-09 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 6: (5 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-09 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 6: (2 comments) Your code looks good but I'd really

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-06 Thread Bikramjeet Vig (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8196 to look at the new patch set (#6). Change subject: IMPALA-4236: Codegen CopyRows() for select nodes ..

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-06 Thread Bikramjeet Vig (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8196 to look at the new patch set (#5). Change subject: IMPALA-4236: Codegen CopyRows() for select nodes ..

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-06 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 4: (2 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 4: (2 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-03 Thread Bikramjeet Vig (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8196 to look at the new patch set (#4). Change subject: IMPALA-4236: Codegen CopyRows() for select nodes ..

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-03 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 3: (4 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 3: Code-Review+1 (4 comments) I'll give Michael Ho a

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-03 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-03 Thread Bikramjeet Vig (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8196 to look at the new patch set (#3). Change subject: IMPALA-4236: Codegen CopyRows() for select nodes ..

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-03 Thread Bikramjeet Vig (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8196 to look at the new patch set (#2). Change subject: IMPALA-4236: Codegen CopyRows() for select nodes ..

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-03 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 1: (11 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 1: (12 comments)

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

2017-10-02 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8196 Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. IMPALA-4236: Codegen CopyRows() for select nodes Preformance: