[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-21 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/131

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-21 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/131

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-14 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. fe: set classpath using maven dependency resolution This

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-14 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 6: Code-Review+2 Forwarding +2 -- To view, visit http

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 6: Verified+1 -- To view, visit http://gerri

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 6: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has removed a vote on this change. Change subject: fe: set classpath using maven dependency resolution .. Removed Verified-1 by Impala Public Jenkins -- To view, visit http://gerrit.cloudera.org:8080/13185 To unsubs

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 5: Verified+1 Test failure was an unrelated apparent fl

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 5: Verified-1 Build failed: https://jenkins.i

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 5: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.clo

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 5: Code-Review+2 -- To view, visit http://ge

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-09 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 3: Build failed: https://jenkins.impala.io/jo

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 3: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 3: Verified-1 Build failed: https://jenkins.i

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 3: I killed the test job because it appeared to be st

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.clou

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 3: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 2: No Builds Executed -- To view, visit htt

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/13

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-08 Thread Todd Lipcon (Code Review)
Hello Vihang Karajgaonkar, Fredy Wijaya, Tim Armstrong, Joe McDonnell, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13185 to look at the new patch set (#2). Change subject: fe: set classpath using maven dependency resolution ..

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-05-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 1: I think we can +2 once Fredy's comments are addres

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-04-30 Thread Vihang Karajgaonkar (Code Review)
Vihang Karajgaonkar has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-04-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/131

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-04-30 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/1

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-04-30 Thread Vihang Karajgaonkar (Code Review)
Vihang Karajgaonkar has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-04-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 1: Verified+1 -- To view, visit http://gerri

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-04-29 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.clo

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-04-29 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 1: Build Successful https://jenkins.impala.i

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-04-29 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13185 ) Change subject: fe: set classpath using maven dependency resolution .. Patch Set 1: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] fe: set classpath using maven dependency resolution

2019-04-29 Thread Todd Lipcon (Code Review)
Hello Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13185 to review the following change. Change subject: fe: set classpath using maven dependency resolution .. fe: set clas