[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 10: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. IMPALA-4277: Support multiple versions of

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 10: Build started:

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-22 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 10: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-22 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 10: (3 comments) You can safely ignore my

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-22 Thread Philip Zeyliger (Code Review)
Hello Thomas Tauber-Marshall, Fredy Wijaya, Tim Armstrong, Joe McDonnell, Alex Behm, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9716 to look at the new patch set (#9). Change subject: IMPALA-4277: Support multiple versions of Hadoop

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-22 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 8: (5 comments)

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-22 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 8: (7 comments)

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-22 Thread Philip Zeyliger (Code Review)
Hello Thomas Tauber-Marshall, Fredy Wijaya, Tim Armstrong, Joe McDonnell, Alex Behm, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9716 to look at the new patch set (#8). Change subject: IMPALA-4277: Support multiple versions of Hadoop

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-22 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 7: Alex & Fredy: could you take a look at

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-22 Thread Philip Zeyliger (Code Review)
Hello Thomas Tauber-Marshall, Tim Armstrong, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9716 to look at the new patch set (#7). Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-21 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 6: Code-Review+2 The difference between PS5

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-21 Thread Philip Zeyliger (Code Review)
Hello Thomas Tauber-Marshall, Tim Armstrong, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9716 to look at the new patch set (#6). Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 5: Code-Review+2 Seems good to me. I assume

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-21 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 5: Code-Review+1 This all makes sense to me.

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-21 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 4: PS4 is a small change to

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-21 Thread Philip Zeyliger (Code Review)
Hello Thomas Tauber-Marshall, Tim Armstrong, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9716 to look at the new patch set (#4). Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-20 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 2: (5 comments) Thanks for your review Tim!

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9716 ) Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. Patch Set 2: Code-Review+1 (5 comments) I did a pass

[Impala-ASF-CR] IMPALA-4277: Support multiple versions of Hadoop ecosystem

2018-03-19 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9716 Change subject: IMPALA-4277: Support multiple versions of Hadoop ecosystem .. IMPALA-4277: Support multiple versions of Hadoop