[Impala-ASF-CR] IMPALA-966: Attribute type error to the right expression in an insert statement

2019-05-08 Thread Alice Fan (Code Review)
Alice Fan has posted comments on this change. ( http://gerrit.cloudera.org:8080/13050 ) Change subject: IMPALA-966: Attribute type error to the right expression in an insert statement .. Patch Set 8: (3 comments)

[Impala-ASF-CR] IMPALA-966: Attribute type error to the right expression in an insert statement

2019-05-08 Thread Alice Fan (Code Review)
Hello Bharath Vissapragada, Paul Rogers, xiaom...@cloudera.com, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13050 to look at the new patch set (#8). Change subject: IMPALA-966: Attribute type error to the right

[Impala-ASF-CR] expr-test: use gtest parameterization

2019-05-08 Thread Todd Lipcon (Code Review)
Hello Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13289 to look at the new patch set (#2). Change subject: expr-test: use gtest parameterization ..

[Impala-ASF-CR] Allow running backend tests sharded and in parallel

2019-05-08 Thread Todd Lipcon (Code Review)
Hello Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13290 to look at the new patch set (#2). Change subject: Allow running backend tests sharded and in parallel

[Impala-ASF-CR] IMPALA-2029. Implement our own getJNIEnv equivalent

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13275 ) Change subject: IMPALA-2029. Implement our own getJNIEnv equivalent .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7031: Add additional info to query canceled from http endpoint

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12926 ) Change subject: IMPALA-7031: Add additional info to query canceled from http endpoint .. Patch Set 10: Build Failed

[Impala-ASF-CR] IMPALA-7031: Add additional info to query canceled from http endpoint

2019-05-08 Thread Alice Fan (Code Review)
Alice Fan has posted comments on this change. ( http://gerrit.cloudera.org:8080/12926 ) Change subject: IMPALA-7031: Add additional info to query canceled from http endpoint .. Patch Set 9: (9 comments)

[Impala-ASF-CR] IMPALA-7031: Add additional info to query canceled from http endpoint

2019-05-08 Thread Alice Fan (Code Review)
Hello Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12926 to look at the new patch set (#10). Change subject: IMPALA-7031: Add additional info to query canceled from http endpoint

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

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has abandoned this change. ( http://gerrit.cloudera.org:8080/13278 ) Change subject: IMPALA-7176: bump timeout for test_insert_mem_limit .. Abandoned will go with michael's patch -- To view, visit

[Impala-ASF-CR] IMPALA-7176: Increase wait time in test insert mem limit

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13292 ) Change subject: IMPALA-7176: Increase wait time in test_insert_mem_limit .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7176: Increase wait time in test insert mem limit

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13292 ) Change subject: IMPALA-7176: Increase wait time in test_insert_mem_limit .. Patch Set 1: Let's just go with this one -- To view,

[Impala-ASF-CR] IMPALA-7176: Increase wait time in 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/13292 ) Change subject: IMPALA-7176: Increase wait time in test_insert_mem_limit .. Patch Set 1: Build Failed

[Impala-ASF-CR] IMPALA-7176: Increase wait time in test insert mem limit

2019-05-08 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/13292 ) Change subject: IMPALA-7176: Increase wait time in test_insert_mem_limit .. Patch Set 1: Didn't know you were working on the same thing.

[Impala-ASF-CR] IMPALA-966: Attribute type error to the right expression in an insert statement

2019-05-08 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/13050 ) Change subject: IMPALA-966: Attribute type error to the right expression in an insert statement .. Patch Set 7: (4 comments) Looks

[Impala-ASF-CR] IMPALA-8528: Refactor authorization check in AnalysisContext

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13285 ) Change subject: IMPALA-8528: Refactor authorization check in AnalysisContext .. Patch Set 5: Build Successful

[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 2: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-7176: Increase wait time in test insert mem limit

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13292 ) Change subject: IMPALA-7176: Increase wait time in test_insert_mem_limit .. Patch Set 1: I wrote almost the same patch. It looks like

[Impala-ASF-CR] expr-test: use gtest parameterization

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13289 ) Change subject: expr-test: use gtest parameterization .. Patch Set 1: Build Successful

[Impala-ASF-CR] expr-test: use gtest parameterization

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13289 ) Change subject: expr-test: use gtest parameterization .. Patch Set 1: This is pretty mechanical so I'm probably OK to +2 once the

[Impala-ASF-CR] expr-test: use gtest parameterization

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13289 ) Change subject: expr-test: use gtest parameterization .. Patch Set 1: (1 comment) This seems like an improvement. I think the

[Impala-ASF-CR] expr-test: use gtest parameterization

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13289 ) Change subject: expr-test: use gtest parameterization .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-7176: Increase wait time in test insert mem limit

2019-05-08 Thread Michael Ho (Code Review)
Michael Ho has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13292 Change subject: IMPALA-7176: Increase wait time in test_insert_mem_limit .. IMPALA-7176: Increase wait time in test_insert_mem_limit

[Impala-ASF-CR] WIP: Allow running tests sharded and in parallel

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13290 ) Change subject: WIP: Allow running tests sharded and in parallel .. Patch Set 1: Build Successful

[Impala-ASF-CR] IMPALA-8528: Refactor authorization check in AnalysisContext

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13285 ) Change subject: IMPALA-8528: Refactor authorization check in AnalysisContext .. Patch Set 5: Rebased to fix the Maven hang issue. --

[Impala-ASF-CR] IMPALA-8528: Refactor authorization check in AnalysisContext

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/13285 ) Change subject: IMPALA-8528: Refactor authorization check in AnalysisContext .. IMPALA-8528: Refactor authorization check in

[Impala-ASF-CR] IMPALA-8072: addendum: don't require fe rebuild for config

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13288 ) Change subject: IMPALA-8072: addendum: don't require fe rebuild for config .. Patch Set 1: Build Successful

[native-toolchain-CR] Update cmake to 3.14.3

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13286 ) Change subject: Update cmake to 3.14.3 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/13286 To

[Impala-ASF-CR] expr-test: use gtest parameterization

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13289 ) Change subject: expr-test: use gtest parameterization .. Patch Set 1: (1 comment)

[Impala-ASF-CR] expr-test: use gtest parameterization

2019-05-08 Thread Todd Lipcon (Code Review)
Hello Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13289 to review the following change. Change subject: expr-test: use gtest parameterization .. expr-test: use gtest

[Impala-ASF-CR] IMPALA-2029. Implement our own getJNIEnv equivalent

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13275 ) Change subject: IMPALA-2029. Implement our own getJNIEnv equivalent .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] WIP: Allow running tests sharded and in parallel

2019-05-08 Thread Todd Lipcon (Code Review)
Hello Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13290 to review the following change. Change subject: WIP: Allow running tests sharded and in parallel .. WIP: Allow

[Impala-ASF-CR] IMPALA-8528: Refactor authorization check in AnalysisContext

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13285 ) Change subject: IMPALA-8528: Refactor authorization check in AnalysisContext .. Patch Set 2: Build Failed

[Impala-ASF-CR] IMPALA-8072: addendum: don't require fe rebuild for config

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13288 ) Change subject: IMPALA-8072: addendum: don't require fe rebuild for config .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-8072: addendum: don't require fe rebuild for config

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13288 Change subject: IMPALA-8072: addendum: don't require fe rebuild for config .. IMPALA-8072: addendum: don't require fe rebuild for

[Impala-ASF-CR] IMPALA-2029. Implement our own getJNIEnv equivalent

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13275 ) Change subject: IMPALA-2029. Implement our own getJNIEnv equivalent .. Patch Set 2: Build Successful

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

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13207 ) Change subject: IMPALA-8460: Simplify cluster membership management .. Patch Set 8: Build Failed

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

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13283 ) Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. Patch Set 3: Verified-1 Build failed:

[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 3: Build started:

[Impala-ASF-CR] IMPALA-8528: Refactor authorization check in AnalysisContext

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13285 ) Change subject: IMPALA-8528: Refactor authorization check in AnalysisContext .. Patch Set 2: I terminated clang-tidy-ub1604 because

[Impala-ASF-CR] IMPALA-8528: Refactor authorization check in AnalysisContext

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13285 ) Change subject: IMPALA-8528: Refactor authorization check in AnalysisContext .. Patch Set 1: Build Failed

[Impala-ASF-CR] Drop statestore update frequency during data loading

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13260 ) Change subject: Drop statestore update frequency during data loading .. Patch Set 3: Verified-1 Build failed:

[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:

[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

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

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13283 ) Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. Patch Set 3: I killed the test job because it appeared to

[Impala-ASF-CR] Drop statestore update frequency during data loading

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13260 ) Change subject: Drop statestore update frequency during data loading .. Patch Set 3: I killed the test job because it appeared to be

[Impala-ASF-CR] IMPALA-966: Attribute type error to the right expression in an insert statement

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13050 ) Change subject: IMPALA-966: Attribute type error to the right expression in an insert statement .. Patch Set 7: Build

[Impala-ASF-CR] IMPALA-2029. Implement our own getJNIEnv equivalent

2019-05-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13275 ) Change subject: IMPALA-2029. Implement our own getJNIEnv equivalent .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-2029. Implement our own getJNIEnv equivalent

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/13275 to look at the new patch set (#2). Change subject: IMPALA-2029. Implement our own getJNIEnv equivalent

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

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13207 ) Change subject: IMPALA-8460: Simplify cluster membership management .. Patch Set 7: Build Failed

[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 2: Build started:

[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 2: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-966: Attribute type error to the right expression in an insert statement

2019-05-08 Thread Alice Fan (Code Review)
Hello Bharath Vissapragada, Paul Rogers, xiaom...@cloudera.com, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13050 to look at the new patch set (#7). Change subject: IMPALA-966: Attribute type error to the right

[Impala-ASF-CR] IMPALA-966: Attribute type error to the right expression in an insert statement

2019-05-08 Thread Alice Fan (Code Review)
Alice Fan has posted comments on this change. ( http://gerrit.cloudera.org:8080/13050 ) Change subject: IMPALA-966: Attribute type error to the right expression in an insert statement .. Patch Set 6: (6 comments)

[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-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: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-8528: Refactor authorization check in AnalysisContext

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/13285 ) Change subject: IMPALA-8528: Refactor authorization check in AnalysisContext .. IMPALA-8528: Refactor authorization check in

[Impala-ASF-CR] [IMPALA-8281] Sentry frontend decoupling

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13284 ) Change subject: [IMPALA-8281] Sentry frontend decoupling .. Patch Set 3: (8 comments)

[native-toolchain-CR] Update cmake to 3.14.3

2019-05-08 Thread Todd Lipcon (Code Review)
Hello Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13286 to review the following change. Change subject: Update cmake to 3.14.3 .. Update cmake to 3.14.3 This updates to

[Impala-ASF-CR] [DOCS] Fixed more typos for SHOW SQL statements

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13273 ) Change subject: [DOCS] Fixed more typos for SHOW SQL statements .. Patch Set 1: (2 comments)

[Impala-ASF-CR] [IMPALA-8281] Sentry frontend decoupling

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13284 ) Change subject: [IMPALA-8281] Sentry frontend decoupling .. Patch Set 2: Build Failed

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

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

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

2019-05-08 Thread Lars Volker (Code Review)
Hello Michael Ho, Thomas Marshall, Tim Armstrong, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13207 to look at the new patch set (#8). Change subject: IMPALA-8460: Simplify cluster membership management

[Impala-ASF-CR] IMPALA-4658: Potential race if compiler reorders ReachedLimit() usage.

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13178 ) Change subject: IMPALA-4658: Potential race if compiler reorders ReachedLimit() usage. .. Patch Set 7: Build Successful

[Impala-ASF-CR] [IMPALA-8281] Sentry frontend decoupling

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13284 ) Change subject: [IMPALA-8281] Sentry frontend decoupling .. Patch Set 3: Build Failed

[Impala-ASF-CR] IMPALA-8528: Refactor authorization check in AnalysisContext

2019-05-08 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13285 Change subject: IMPALA-8528: Refactor authorization check in AnalysisContext .. IMPALA-8528: Refactor authorization check in

[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

[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: Verified-1 Build failed:

[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: Verified-1 Build failed:

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

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13207 ) Change subject: IMPALA-8460: Simplify cluster membership management .. Patch Set 7: (3 comments) Had a few comments, still need to

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

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

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

2019-05-08 Thread Lars Volker (Code Review)
Hello Michael Ho, Thomas Marshall, Tim Armstrong, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13207 to look at the new patch set (#7). Change subject: IMPALA-8460: Simplify cluster membership management

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

2019-05-08 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/13207 ) Change subject: IMPALA-8460: Simplify cluster membership management .. Patch Set 7: Code-Review+1 Carrying Bikram's +1 -- To view,

[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:

[Impala-ASF-CR] IMPALA-4658: Potential race if compiler reorders ReachedLimit() usage.

2019-05-08 Thread Abhishek Rawat (Code Review)
Abhishek Rawat has uploaded a new patch set (#7). ( http://gerrit.cloudera.org:8080/13178 ) Change subject: IMPALA-4658: Potential race if compiler reorders ReachedLimit() usage. .. IMPALA-4658: Potential race if compiler

[Impala-ASF-CR] IMPALA-2029. Implement our own getJNIEnv equivalent

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13275 ) Change subject: IMPALA-2029. Implement our own getJNIEnv equivalent .. Patch Set 1: Code-Review+2 (1 comment) This seems to make

[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: Verified+1 It was still somewhat slow in the download

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

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13279 ) Change subject: IMPALA-8527: prune maven repo dependencies .. IMPALA-8527: prune maven repo dependencies We transitively pull

[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 Successful

[Impala-ASF-CR] [IMPALA-8281] Sentry frontend decoupling

2019-05-08 Thread Austin Nobis (Code Review)
Hello Fredy Wijaya, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13284 to look at the new patch set (#3). Change subject: [IMPALA-8281] Sentry frontend decoupling

[Impala-ASF-CR] [IMPALA-8281] Sentry frontend decoupling

2019-05-08 Thread Austin Nobis (Code Review)
Austin Nobis has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13284 Change subject: [IMPALA-8281] Sentry frontend decoupling .. [IMPALA-8281] Sentry frontend decoupling This patch moves Sentry specific

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

2019-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong 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: The job was stuck because of the maven

[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: Build Failed

[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: Verified-1 Build failed:

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

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13283 ) Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. Patch Set 3: Build Failed

[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

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

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13283 ) Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. Patch Set 2: Build Failed

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

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13283 ) Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. Patch Set 1: Build Failed

[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 4: Verified-1 Build failed:

[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 3: Verified-1 Build failed:

[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)

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

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13283 ) Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. Patch Set 3: Build started:

[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] IMPALA-8369: Fixing some core tests in Hive environment

2019-05-08 Thread Vihang Karajgaonkar (Code Review)
Vihang Karajgaonkar has posted comments on this change. ( http://gerrit.cloudera.org:8080/13283 ) Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. Patch Set 3: (2 comments)

[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:

[Impala-ASF-CR] Drop statestore update frequency during data loading

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13260 ) Change subject: Drop statestore update frequency during data loading .. Patch Set 3: Build started:

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

2019-05-08 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/13283 ) Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. Patch Set 3: (1 comment)

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

2019-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13283 ) Change subject: IMPALA-8369: Fixing some core tests in Hive environment .. Patch Set 2: (1 comment)

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

2019-05-08 Thread Todd Lipcon (Code Review)
Todd Lipcon 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+2 Let's get this in as soon as we see the

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

2019-05-08 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has uploaded a new patch set (#2). ( 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-8369: Fixing some core tests in Hive environment

2019-05-08 Thread Csaba Ringhofer (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13283 to look at the new patch set (#3). Change subject: IMPALA-8369: Fixing some core tests in Hive environment

  1   2   >