[Impala-ASF-CR] IMPALA-8369: Fixing some core tests in Hive environment

2019-05-08 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13283 Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. IMPALA-8369: Fixing some core tests in Hive environment

[Impala-ASF-CR] IMPALA-8527: prune maven repo dependencies

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: IMPALA-8527: prune maven repo dependencies .. Patch Set 5: Build Successful

[Impala-ASF-CR] IMPALA-7608: Estimate row count from file size when no stats available

2019-05-08 Thread Fang-Yu Rao (Code Review)
Fang-Yu Rao has removed Todd Lipcon from this change. ( http://gerrit.cloudera.org:8080/12974 ) Change subject: IMPALA-7608: Estimate row count from file size when no stats available .. Removed reviewer Todd Lipcon. -- To

[Impala-ASF-CR] IMPALA-8527: prune maven repo dependencies

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: IMPALA-8527: prune maven repo dependencies .. Patch Set 6: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-8524: Avoid calling "hive" via command line in EE tests

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13282 ) Change subject: IMPALA-8524: Avoid calling "hive" via command line in EE tests .. Patch Set 1: Build Failed

[Impala-ASF-CR] IMPALA-8369 (part 2): Hive 3: switch to Tez-on-YARN execution

2019-05-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13224 ) Change subject: IMPALA-8369 (part 2): Hive 3: switch to Tez-on-YARN execution .. Patch Set 7: > Patch Set 7: > > I ran this (without

[Impala-ASF-CR] IMPALA-8512: Disable certain tests on Centos6

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13271 ) Change subject: IMPALA-8512: Disable certain tests on Centos6 .. Patch Set 4: Verified+1 I'm just going to override jenkins on this

[Impala-ASF-CR] IMPALA-7176: bump timeout for test insert mem limit

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13278 ) Change subject: IMPALA-7176: bump timeout for test_insert_mem_limit .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-8517: print backtrace to debug bootstrap toolchain

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13270 ) Change subject: IMPALA-8517: print backtrace to debug bootstrap_toolchain .. IMPALA-8517: print backtrace to debug

[Impala-ASF-CR] IMPALA-8517: print backtrace to debug bootstrap toolchain

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13270 ) Change subject: IMPALA-8517: print backtrace to debug bootstrap_toolchain .. Patch Set 4: Verified+1 Only one job timed out because

[Impala-ASF-CR] IMPALA-8517: print backtrace to debug bootstrap toolchain

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13270 ) Change subject: IMPALA-8517: print backtrace to debug bootstrap_toolchain .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-8460: Simplify cluster membership management

2019-05-08 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/13207 ) Change subject: IMPALA-8460: Simplify cluster membership management .. Patch Set 6: Code-Review+1 (5 comments)

[Impala-ASF-CR] IMPALA-8512: Disable certain tests on Centos6

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has removed a vote on this change. Change subject: IMPALA-8512: Disable certain tests on Centos6 .. Removed Verified-1 by Impala Public Jenkins -- To view, visit http://gerrit.cloudera.org:8080/13271 To

[Impala-ASF-CR] IMPALA-8512: Disable certain tests on Centos6

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13271 ) Change subject: IMPALA-8512: Disable certain tests on Centos6 .. IMPALA-8512: Disable certain tests on Centos6 The data cache

[Impala-ASF-CR] IMPALA-8516. Update maven for Jenkins builds

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13268 ) Change subject: IMPALA-8516. Update maven for Jenkins builds .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-8517: print backtrace to debug bootstrap toolchain

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has removed a vote on this change. Change subject: IMPALA-8517: print backtrace to debug bootstrap_toolchain .. Removed Verified-1 by Impala Public Jenkins -- To view, visit http://gerrit.cloudera.org:8080/13270

[Impala-ASF-CR] IMPALA-8516. Update maven for Jenkins builds

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13268 ) Change subject: IMPALA-8516. Update maven for Jenkins builds .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-8527: prune maven repo dependencies

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: IMPALA-8527: prune maven repo dependencies .. Patch Set 6: Build started:

[Impala-ASF-CR] IMPALA-8527: prune maven repo dependencies

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: IMPALA-8527: prune maven repo dependencies .. Patch Set 6: It looks like PS4 avoided getting stuck, so I cleaned this

[Impala-ASF-CR] IMPALA-8527: prune maven repo dependencies

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: IMPALA-8527: prune maven repo dependencies .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-8527: prune maven repo dependencies

2019-05-08 Thread Tim Armstrong (Code Review)
Hello Fredy Wijaya, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13279 to look at the new patch set (#6). Change subject: IMPALA-8527: prune maven repo dependencies

[Impala-ASF-CR] IMPALA-8512: Disable certain tests on Centos6

2019-05-08 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/13271 ) Change subject: IMPALA-8512: Disable certain tests on Centos6 .. Patch Set 4: GVO failed due to IMPALA-8527 -- To view, visit

[Impala-ASF-CR] IMPALA-8527: prune maven repo dependencies

2019-05-08 Thread Tim Armstrong (Code Review)
Hello Fredy Wijaya, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13279 to look at the new patch set (#5). Change subject: IMPALA-8527: prune maven repo dependencies

[Impala-ASF-CR] IMPALA-8516. Update maven for Jenkins builds

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13268 ) Change subject: IMPALA-8516. Update maven for Jenkins builds .. Patch Set 2: (4 comments)

[Impala-ASF-CR] IMPALA-8516. Update maven for Jenkins builds

2019-05-08 Thread Todd Lipcon (Code Review)
Hello Tim Armstrong, Joe McDonnell, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13268 to look at the new patch set (#2). Change subject: IMPALA-8516. Update maven for Jenkins builds

[Impala-ASF-CR] IMPALA-8516. Update maven for Jenkins builds

2019-05-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13268 ) Change subject: IMPALA-8516. Update maven for Jenkins builds .. Patch Set 1: Apparently Maven 3.6.1 doesn't build Impala because it

[Impala-ASF-CR] IMPALA-8524: Avoid calling "hive" via command line in EE tests

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13282 ) Change subject: IMPALA-8524: Avoid calling "hive" via command line in EE tests .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-8524: Avoid calling "hive" via command line in EE tests

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13282 ) Change subject: IMPALA-8524: Avoid calling "hive" via command line in EE tests .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-8524: Avoid calling "hive" via command line in EE tests

2019-05-08 Thread Yongzhi Chen (Code Review)
Yongzhi Chen has posted comments on this change. ( http://gerrit.cloudera.org:8080/13282 ) Change subject: IMPALA-8524: Avoid calling "hive" via command line in EE tests .. Patch Set 1: Code-Review+1 Hive Cli is deprecated.

[Impala-ASF-CR] WIP ONLY - Try to reduce number of MVN repo dependencies

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: WIP ONLY - Try to reduce number of MVN repo dependencies .. Patch Set 3: (1 comment)

[Impala-ASF-CR] WIP ONLY - Try to reduce number of MVN repo dependencies

2019-05-08 Thread Tim Armstrong (Code Review)
Hello Fredy Wijaya, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13279 to look at the new patch set (#4). Change subject: WIP ONLY - Try to reduce number of MVN repo dependencies

[Impala-ASF-CR] WIP ONLY - Try to reduce number of MVN repo dependencies

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: WIP ONLY - Try to reduce number of MVN repo dependencies .. Patch Set 4: Build started:

[Impala-ASF-CR] WIP ONLY - Try to reduce number of MVN repo dependencies

2019-05-08 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13279 to look at the new patch set (#3). Change subject: WIP ONLY - Try to reduce number of MVN repo dependencies

[Impala-ASF-CR] IMPALA-8369 (part 4): Hive 3: fixes for functional dataset loading

2019-05-08 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13251 ) Change subject: IMPALA-8369 (part 4): Hive 3: fixes for functional dataset loading .. Patch Set 3: (1 comment) What happened to the

[Impala-ASF-CR] WIP ONLY - Try to reduce number of MVN repo dependencies

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: WIP ONLY - Try to reduce number of MVN repo dependencies .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-8524: Avoid calling "hive" via command line in EE tests

2019-05-08 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13282 Change subject: IMPALA-8524: Avoid calling "hive" via command line in EE tests .. IMPALA-8524: Avoid calling "hive" via command line

[Impala-ASF-CR] WIP ONLY - Try to reduce number of MVN repo dependencies

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: WIP ONLY - Try to reduce number of MVN repo dependencies .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-8512: Disable certain tests on Centos6

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13271 ) Change subject: IMPALA-8512: Disable certain tests on Centos6 .. Patch Set 4: Build failed:

[Impala-ASF-CR] IMPALA-8369 (part 4): Hive 3: fixes for functional dataset loading

2019-05-08 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13251 ) Change subject: IMPALA-8369 (part 4): Hive 3: fixes for functional dataset loading .. Patch Set 3: (2 comments) Forgot to post this

[Impala-ASF-CR] IMPALA-8509. Lazily evaluate LOAD sections during data load

2019-05-08 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/13252 ) Change subject: IMPALA-8509. Lazily evaluate LOAD sections during data load .. Patch Set 3: Code-Review+2 (1 comment) I think this

[Impala-ASF-CR] IMPALA-8121: part 2: use local catalog in containers

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13226 ) Change subject: IMPALA-8121: part 2: use local catalog in containers .. Patch Set 9: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-5843: Use page index in Parquet files to skip pages

2019-05-08 Thread Zoltan Borok-Nagy (Code Review)
Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12065 ) Change subject: IMPALA-5843: Use page index in Parquet files to skip pages .. Patch Set 19: (1 comment)

[Impala-ASF-CR] IMPALA-5843: Use page index in Parquet files to skip pages

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12065 ) Change subject: IMPALA-5843: Use page index in Parquet files to skip pages .. Patch Set 19: Build started:

[Impala-ASF-CR] IMPALA-5843: Use page index in Parquet files to skip pages

2019-05-08 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/12065 ) Change subject: IMPALA-5843: Use page index in Parquet files to skip pages .. Patch Set 19: Code-Review+2 I am ok with the current

[Impala-ASF-CR] IMPALA-7293: Show tuple layout in explain plan.

2019-05-08 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/13193 ) Change subject: IMPALA-7293: Show tuple layout in explain plan. .. Patch Set 4: Code-Review+1 (4 comments) I have a few more

[Impala-ASF-CR] IMPALA-8517: print backtrace to debug bootstrap toolchain

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13270 ) Change subject: IMPALA-8517: print backtrace to debug bootstrap_toolchain .. Patch Set 4: Verified-1 Build failed:

<    1   2