[Impala-ASF-CR] IMPALA-3815: clean up cross-compiled comparator

2016-09-08 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3815: clean up cross-compiled comparator .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4307 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3815: clean up cross-compiled comparator

2016-09-08 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3815: clean up cross-compiled comparator .. IMPALA-3815: clean up cross-compiled comparator This avoids including a big block of interpreted

[Impala-ASF-CR] IMPALA-3815: clean up cross-compiled comparator

2016-09-08 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3815: clean up cross-compiled comparator .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4307 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3815: clean up cross-compiled comparator

2016-09-08 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Internal Jenkins, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4307 to look at the new patch set (#3). Change subject: IMPALA-3815: clean up cross-compiled comparator

[Impala-ASF-CR] IMPALA-3815: clean up cross-compiled comparator

2016-09-08 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3815: clean up cross-compiled comparator .. Patch Set 2: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-3815: clean up cross-compiled comparator

2016-09-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3815: clean up cross-compiled comparator .. Patch Set 2: Code-Review+2 Carry +2 -- To view, visit http://gerrit.cloudera.org:8080/4307 To unsubscribe,

[Impala-ASF-CR] IMPALA-3815: clean up cross-compiled comparator

2016-09-07 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3815: clean up cross-compiled comparator .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4307 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3815: clean up cross-compiled comparator

2016-09-06 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-3815: clean up cross-compiled comparator .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4307 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3815: clean up cross-compiled comparator

2016-09-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/4307 Change subject: IMPALA-3815: clean up cross-compiled comparator .. IMPALA-3815: clean up cross-compiled comparator This avoids