[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12078 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12078 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Remove Python 2.4 workarounds in

[Impala-ASF-CR] IMPALA-7915: Wrap SQL parser to avoid redundant code

2018-12-12 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/12016 ) Change subject: IMPALA-7915: Wrap SQL parser to avoid redundant code .. Patch Set 3: That looks related to your change.

[Impala-ASF-CR] IMPALA-7351 IMPALA-7353: Add justification for constant memory estimates

2018-12-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12063 ) Change subject: IMPALA-7351 IMPALA-7353: Add justification for constant memory estimates .. Patch Set 1: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-7867 (Part 2): ArrayList cleanup in analyzer

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11995 ) Change subject: IMPALA-7867 (Part 2): ArrayList cleanup in analyzer .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-5474: Adding a trivial subquery turns error into warning

2018-12-12 Thread Yongjun Zhang (Code Review)
Yongjun Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12022 ) Change subject: IMPALA-5474: Adding a trivial subquery turns error into warning .. Patch Set 4: > Build failed:

[Impala-ASF-CR] IMPALA-7902: NumericLiteral fixes, refactoring

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12001 ) Change subject: IMPALA-7902: NumericLiteral fixes, refactoring .. Patch Set 11: Build Successful

[Impala-ASF-CR] IMPALA-7867 (Part 2): ArrayList cleanup in analyzer

2018-12-12 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/11995 ) Change subject: IMPALA-7867 (Part 2): ArrayList cleanup in analyzer .. Patch Set 3: Thanks much for the reviews. Rebased on latest

[Impala-ASF-CR] IMPALA-7915: Wrap SQL parser to avoid redundant code

2018-12-12 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/12016 ) Change subject: IMPALA-7915: Wrap SQL parser to avoid redundant code .. Patch Set 3: Failure seems to be in one of the Python tests:

[Impala-ASF-CR] IMPALA-7902: NumericLiteral fixes, refactoring

2018-12-12 Thread Paul Rogers (Code Review)
Hello Bharath Vissapragada, Fredy Wijaya, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12001 to look at the new patch set (#11). Change subject: IMPALA-7902: NumericLiteral fixes, refactoring

[Impala-ASF-CR] IMPALA-7351: Add estimates to kudu table sink

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12077 ) Change subject: IMPALA-7351: Add estimates to kudu table sink .. IMPALA-7351: Add estimates to kudu table sink The

[Impala-ASF-CR] IMPALA-7351: Add estimates to kudu table sink

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12077 ) Change subject: IMPALA-7351: Add estimates to kudu table sink .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 5: Build Successful

[Impala-ASF-CR] IMPALA-5474: Adding a trivial subquery turns error into warning

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12022 ) Change subject: IMPALA-5474: Adding a trivial subquery turns error into warning .. Patch Set 4: Verified-1 Build failed:

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12078 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 4: (8 comments)

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-12 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12074 to look at the new patch set (#5). Change subject: IMPALA-7948: part 1: initial docker container build

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12078 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7844: HAVING clause cannot support ordinals

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11955 ) Change subject: IMPALA-7844: HAVING clause cannot support ordinals .. Patch Set 5: Build Successful

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12078 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 2: Build Successful

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12078 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12078 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 1: Build Successful

[Impala-ASF-CR] IMPALA-7844: HAVING clause cannot support ordinals

2018-12-12 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/11955 ) Change subject: IMPALA-7844: HAVING clause cannot support ordinals .. Patch Set 4: (10 comments) Revised based on review comments.

[Impala-ASF-CR] IMPALA-7844: HAVING clause cannot support ordinals

2018-12-12 Thread Paul Rogers (Code Review)
Hello Bharath Vissapragada, Greg Rahn, Zoltan Borok-Nagy, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11955 to look at the new patch set (#5). Change subject: IMPALA-7844: HAVING clause cannot support ordinals

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Tim Armstrong (Code Review)
Hello Lars Volker, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12078 to look at the new patch set (#2). Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12078 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 1: (2 comments)

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12078 ) Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Patch Set 1: (2 comments)

[Impala-ASF-CR] Remove Python 2.4 workarounds in start-impala-cluster.py

2018-12-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12078 Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py .. Remove Python 2.4 workarounds in start-impala-cluster.py

[Impala-ASF-CR] IMPALA-7915: Wrap SQL parser to avoid redundant code

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12016 ) Change subject: IMPALA-7915: Wrap SQL parser to avoid redundant code .. Patch Set 3: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-7926: Fix flakiness in test reconnect

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12045 ) Change subject: IMPALA-7926: Fix flakiness in test_reconnect .. IMPALA-7926: Fix flakiness in test_reconnect

[Impala-ASF-CR] IMPALA-7926: Fix flakiness in test reconnect

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12045 ) Change subject: IMPALA-7926: Fix flakiness in test_reconnect .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7657: Codegen IsNotEmptyPredicate and ValidTupleIdExpr.

2018-12-12 Thread Andrew Sherman (Code Review)
Andrew Sherman has posted comments on this change. ( http://gerrit.cloudera.org:8080/12068 ) Change subject: IMPALA-7657: Codegen IsNotEmptyPredicate and ValidTupleIdExpr. .. Patch Set 3: I tested performance of the two

[Impala-ASF-CR] IMPALA-7795: Implement REFRESH AUTHORIZATION statement

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11888 ) Change subject: IMPALA-7795: Implement REFRESH AUTHORIZATION statement .. Patch Set 12: Build Successful

[Impala-ASF-CR] IMPALA-7902: NumericLiteral fixes, refactoring

2018-12-12 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/12001 ) Change subject: IMPALA-7902: NumericLiteral fixes, refactoring .. Patch Set 10: Code-Review+1 (3 comments) The code flow

[Impala-ASF-CR] IMPALA-7351: Add estimates to kudu table sink

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12077 ) Change subject: IMPALA-7351: Add estimates to kudu table sink .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7351: Add estimates to kudu table sink

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12077 ) Change subject: IMPALA-7351: Add estimates to kudu table sink .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7795: Implement REFRESH AUTHORIZATION statement

2018-12-12 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/11888 ) Change subject: IMPALA-7795: Implement REFRESH AUTHORIZATION statement .. Patch Set 12: (7 comments)

[Impala-ASF-CR] IMPALA-7795: Implement REFRESH AUTHORIZATION statement

2018-12-12 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#12). ( http://gerrit.cloudera.org:8080/11888 ) Change subject: IMPALA-7795: Implement REFRESH AUTHORIZATION statement .. IMPALA-7795: Implement REFRESH AUTHORIZATION statement This

[Impala-ASF-CR] IMPALA-7351: Add estimates to kudu table sink

2018-12-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12077 ) Change subject: IMPALA-7351: Add estimates to kudu table sink .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-5474: Adding a trivial subquery turns error into warning

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12022 ) Change subject: IMPALA-5474: Adding a trivial subquery turns error into warning .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-5474: Adding a trivial subquery turns error into warning

2018-12-12 Thread Yongjun Zhang (Code Review)
Yongjun Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12022 ) Change subject: IMPALA-5474: Adding a trivial subquery turns error into warning .. Patch Set 4: Thanks much for the review and

[Impala-ASF-CR] IMPALA-5474: Adding a trivial subquery turns error into warning

2018-12-12 Thread Yongjun Zhang (Code Review)
Yongjun Zhang has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/12022 ) Change subject: IMPALA-5474: Adding a trivial subquery turns error into warning .. IMPALA-5474: Adding a trivial subquery turns error

[Impala-ASF-CR] IMPALA-6591: Fix test ssl flaky test

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12047 ) Change subject: IMPALA-6591: Fix test_ssl flaky test .. Patch Set 8: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6591: Fix test ssl flaky test

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12047 ) Change subject: IMPALA-6591: Fix test_ssl flaky test .. IMPALA-6591: Fix test_ssl flaky test test_ssl has a logic

[Impala-ASF-CR] IMPALA-7351: Add estimates to kudu table sink

2018-12-12 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/12077 ) Change subject: IMPALA-7351: Add estimates to kudu table sink .. Patch Set 1: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7047. Refreshing partitions should not make an RPC per file

2018-12-12 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/11227 ) Change subject: IMPALA-7047. Refreshing partitions should not make an RPC per file .. Patch Set 7: Had a chat with Paul

[Impala-ASF-CR] IMPALA-7351 IMPALA-7353: Add justification for constant memory estimates

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12063 ) Change subject: IMPALA-7351 IMPALA-7353: Add justification for constant memory estimates .. Patch Set 1: Build Successful

[Impala-ASF-CR] IMPALA-7351: Add estimates to kudu table sink

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12077 ) Change subject: IMPALA-7351: Add estimates to kudu table sink .. Patch Set 1: Build Successful

[Impala-ASF-CR] WIP IMPALA-7178: Add the possibility to reduce logging for common data errors

2018-12-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/10793 ) Change subject: WIP IMPALA-7178: Add the possibility to reduce logging for common data errors .. Patch Set 1: Any updates? Should we

[Impala-ASF-CR] [WIP] IMPALA-6932: Speed up scans for sequence datasets with many files

2018-12-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/11517 ) Change subject: [WIP] IMPALA-6932: Speed up scans for sequence datasets with many files .. Patch Set 3: The discussion looks like it's

[Impala-ASF-CR] IMPALA-6964: Track stats about column and page sizes in Parquet reader

2018-12-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/11575 ) Change subject: IMPALA-6964: Track stats about column and page sizes in Parquet reader .. Patch Set 14: (12 comments)

[Impala-ASF-CR] IMPALA-7915: Wrap SQL parser to avoid redundant code

2018-12-12 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/12016 ) Change subject: IMPALA-7915: Wrap SQL parser to avoid redundant code .. Patch Set 2: Code-Review+2 Thanks for the clean up.

[Impala-ASF-CR] IMPALA-7915: Wrap SQL parser to avoid redundant code

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12016 ) Change subject: IMPALA-7915: Wrap SQL parser to avoid redundant code .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-7915: Wrap SQL parser to avoid redundant code

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12016 ) Change subject: IMPALA-7915: Wrap SQL parser to avoid redundant code .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7351 IMPALA-7353: Add justification for constant memory estimates

2018-12-12 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12063 Change subject: IMPALA-7351 IMPALA-7353: Add justification for constant memory estimates .. IMPALA-7351 IMPALA-7353: Add

[Impala-ASF-CR] IMPALA-7351: Add estimates to kudu table sink

2018-12-12 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12077 Change subject: IMPALA-7351: Add estimates to kudu table sink .. IMPALA-7351: Add estimates to kudu table sink The kudu table sink

[Impala-ASF-CR] IMPALA-7926: Fix flakiness in test reconnect

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12045 ) Change subject: IMPALA-7926: Fix flakiness in test_reconnect .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7926: Fix flakiness in test reconnect

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12045 ) Change subject: IMPALA-7926: Fix flakiness in test_reconnect .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-7926: Fix flakiness in test reconnect

2018-12-12 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/12045 ) Change subject: IMPALA-7926: Fix flakiness in test_reconnect .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7694: Add host resource usage metrics to profile

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12069 ) Change subject: IMPALA-7694: Add host resource usage metrics to profile .. Patch Set 7: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7948: part 1: initial docker container build

2018-12-12 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12074 ) Change subject: IMPALA-7948: part 1: initial docker container build .. Patch Set 4: (8 comments) Overall structure seems fine. See

[Impala-ASF-CR] IMPALA-7625: test web pages.py backend tests are failing

2018-12-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12024 ) Change subject: IMPALA-7625: test_web_pages.py backend tests are failing .. Patch Set 4: Code-Review+1 (2 comments) I only had some

[Impala-ASF-CR] IMPALA-7924: Generate Thrift 11 Python Code

2018-12-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12036 ) Change subject: IMPALA-7924: Generate Thrift 11 Python Code .. Patch Set 1: Code-Review+1 (4 comments)

[Impala-ASF-CR] IMPALA-7807: Analysis test fixture

2018-12-12 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/11881 ) Change subject: IMPALA-7807: Analysis test fixture .. Patch Set 10: (4 comments)

[Impala-ASF-CR] IMPALA-6591: Fix test ssl flaky test

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12047 ) Change subject: IMPALA-6591: Fix test_ssl flaky test .. Patch Set 7: Build Successful

[Impala-ASF-CR] IMPALA-6591: Fix test ssl flaky test

2018-12-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12047 ) Change subject: IMPALA-6591: Fix test_ssl flaky test .. Patch Set 7: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/12071 ) Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Alex Rodoni (Code Review)
Alex Rodoni has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12071 ) Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges .. IMPALA-7728: [DOCS] Added a section on Changing

[Impala-ASF-CR] IMPALA-6591: Fix test ssl flaky test

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12047 ) Change subject: IMPALA-6591: Fix test_ssl flaky test .. Patch Set 8: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12071 ) Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges .. Patch Set 3: Verified+1 Build Successful

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12071 ) Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges .. Patch Set 2: Verified+1 Build Successful

[native-toolchain-CR] IMPALA-7924: Add Thrift 0.11.0 to the toolchain

2018-12-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12034 ) Change subject: IMPALA-7924: Add Thrift 0.11.0 to the toolchain .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7926: Fix flakiness in test reconnect

2018-12-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12045 ) Change subject: IMPALA-7926: Fix flakiness in test_reconnect .. Patch Set 2: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12071 ) Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges .. Patch Set 3: Build Started

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Alex Rodoni (Code Review)
Hello Fredy Wijaya, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12071 to look at the new patch set (#3). Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12071 ) Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Alex Rodoni (Code Review)
Hello Fredy Wijaya, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12071 to look at the new patch set (#2). Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12071 ) Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges .. Patch Set 2: Build Started

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/12071 ) Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7728: [DOCS] Added a section on Changing Privileges

2018-12-12 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12071 ) Change subject: IMPALA-7728: [DOCS] Added a section on Changing Privileges .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6591: Fix test ssl flaky test

2018-12-12 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12047 ) Change subject: IMPALA-6591: Fix test_ssl flaky test .. Patch Set 7: (3 comments)

[Impala-ASF-CR] IMPALA-6591: Fix test ssl flaky test

2018-12-12 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#7). ( http://gerrit.cloudera.org:8080/12047 ) Change subject: IMPALA-6591: Fix test_ssl flaky test .. IMPALA-6591: Fix test_ssl flaky test test_ssl has a logic that waits for the

[Impala-ASF-CR] IMPALA-7867 (Part 2): ArrayList cleanup in analyzer

2018-12-12 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/11995 ) Change subject: IMPALA-7867 (Part 2): ArrayList cleanup in analyzer .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7867 (Part 2): ArrayList cleanup in analyzer

2018-12-12 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/11995 ) Change subject: IMPALA-7867 (Part 2): ArrayList cleanup in analyzer .. Patch Set 3: Code-Review+1 Fredy, could you +2 when you

[Impala-ASF-CR] IMPALA-7844: HAVING clause cannot support ordinals

2018-12-12 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/11955 ) Change subject: IMPALA-7844: HAVING clause cannot support ordinals .. Patch Set 4: Hmm. I've been thinking about it and since

[Impala-ASF-CR] Update version to 3.2.0-SNAPSHOT

2018-12-12 Thread Zoltan Borok-Nagy (Code Review)
Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12053 ) Change subject: Update version to 3.2.0-SNAPSHOT .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] Update version to 3.2.0-SNAPSHOT

2018-12-12 Thread Zoltan Borok-Nagy (Code Review)
Zoltan Borok-Nagy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12053 ) Change subject: Update version to 3.2.0-SNAPSHOT .. Update version to 3.2.0-SNAPSHOT Change-Id:

[Impala-ASF-CR] IMPALA-7694: Add host resource usage metrics to profile

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12069 ) Change subject: IMPALA-7694: Add host resource usage metrics to profile .. Patch Set 7: Build started:

[Impala-ASF-CR] IMPALA-7844: HAVING clause cannot support ordinals

2018-12-12 Thread Zoltan Borok-Nagy (Code Review)
Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/11955 ) Change subject: IMPALA-7844: HAVING clause cannot support ordinals .. Patch Set 4: (1 comment) I like the direction where this

[Impala-ASF-CR] IMPALA-7694: Add host resource usage metrics to profile

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12069 ) Change subject: IMPALA-7694: Add host resource usage metrics to profile .. Patch Set 7: Build Successful

[Impala-ASF-CR] IMPALA-7694: Add host resource usage metrics to profile

2018-12-12 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12069 ) Change subject: IMPALA-7694: Add host resource usage metrics to profile .. Patch Set 6: Build Successful