[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 12: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 13: Code-Review+2 (3 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#12). Change subject: IMPALA-1048: show sinks in exec summary

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 11: (1 comment)

[Impala-ASF-CR] IMPALA-7842: Expose physical plan for unit testing

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11920 ) Change subject: IMPALA-7842: Expose physical plan for unit testing .. Patch Set 5: Build Successful

[Impala-ASF-CR] IMPALA-6924: Add child queries to profile in compute stats

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11977 ) Change subject: IMPALA-6924: Add child queries to profile in compute stats .. Patch Set 4: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 10: (5 comments)

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Tim Armstrong (Code Review)
Hello Pooja Nilangekar, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10928 to look at the new patch set (#11). Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit

[Impala-ASF-CR] IMPALA-7908: [DOCS] Take 2 More clarifications on the new Admission options

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12028 ) Change subject: IMPALA-7908: [DOCS] Take 2 More clarifications on the new Admission options .. Patch Set 2: Build Started

[Impala-ASF-CR] IMPALA-7908: [DOCS] Take 2 More clarifications on the new Admission options

2018-12-03 Thread Alex Rodoni (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12028 to look at the new patch set (#2). Change subject: IMPALA-7908: [DOCS] Take 2 More clarifications on the new Admission options

[Impala-ASF-CR] IMPALA-7914: Base interface for statement-like AST nodes

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12018 ) Change subject: IMPALA-7914: Base interface for statement-like AST nodes .. Patch Set 3: Build Successful

[Impala-ASF-CR] IMPALA-7914: Base interface for statement-like AST nodes

2018-12-03 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/12018 ) Change subject: IMPALA-7914: Base interface for statement-like AST nodes .. Patch Set 2: Code-Review+2 (4 comments) Nice

[Impala-ASF-CR] IMPALA-2343: Add lifecycle timeline to plan nodes

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11992 ) Change subject: IMPALA-2343: Add lifecycle timeline to plan nodes .. Patch Set 11: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-2343: Add lifecycle timeline to plan nodes

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11992 ) Change subject: IMPALA-2343: Add lifecycle timeline to plan nodes .. IMPALA-2343: Add lifecycle timeline to plan nodes

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 12: Build Successful

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 14: Build Successful

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 10: (4 comments)

[Impala-ASF-CR](2.x) IMPALA-7922: Fix 2.x broken build due to Jackson dependency update

2018-12-03 Thread Quanlong Huang (Code Review)
Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12027 ) Change subject: IMPALA-7922: Fix 2.x broken build due to Jackson dependency update .. Patch Set 4: > Patch Set 3: > > I'm going to

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Tim Armstrong (Code Review)
Hello Pooja Nilangekar, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10928 to look at the new patch set (#13). Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit

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

2018-12-03 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/12001 ) Change subject: IMPALA-7902: NumericLiteral fixes, refactoring .. Patch Set 8: (10 comments) Thanks Tim for your review comments. I've

[Impala-ASF-CR] IMPALA-7914: Base interface for statement-like AST nodes

2018-12-03 Thread Paul Rogers (Code Review)
Hello Bharath Vissapragada, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12018 to look at the new patch set (#3). Change subject: IMPALA-7914: Base interface for statement-like AST nodes

[Impala-ASF-CR] IMPALA-7914: Base interface for statement-like AST nodes

2018-12-03 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/12018 ) Change subject: IMPALA-7914: Base interface for statement-like AST nodes .. Patch Set 2: (3 comments) Bharath, thanks much for your

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12000 ) Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates .. IMPALA-6741: Add timestamp of

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12000 ) Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates .. Patch Set 7: Verified+1 -- To view,

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

2018-12-03 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 (#9). Change subject: IMPALA-7902: NumericLiteral fixes, refactoring

[Impala-ASF-CR] IMPALA-7473: fix crash when printing decimal with -v=3

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12029 ) Change subject: IMPALA-7473: fix crash when printing decimal with -v=3 .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7842: Expose physical plan for unit testing

2018-12-03 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/11920 ) Change subject: IMPALA-7842: Expose physical plan for unit testing .. Patch Set 5: I think you missed a few comments outside

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Pooja Nilangekar (Code Review)
Pooja Nilangekar has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 10: Code-Review+1 (1 comment) Looks

[Impala-ASF-CR](2.x) IMPALA-7922: Fix 2.x broken build due to Jackson dependency update

2018-12-03 Thread Quanlong Huang (Code Review)
Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12027 ) Change subject: IMPALA-7922: Fix 2.x broken build due to Jackson dependency update .. Patch Set 4: > Patch Set 3: > > > Patch Set

[Impala-ASF-CR] IMPALA-7908: [DOCS] More clarifications on the new Admission options

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12028 ) Change subject: IMPALA-7908: [DOCS] More clarifications on the new Admission options .. Patch Set 1: Build Started

[Impala-ASF-CR] IMPALA-7908: [DOCS] More clarifications on the new Admission options

2018-12-03 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12028 Change subject: IMPALA-7908: [DOCS] More clarifications on the new Admission options .. IMPALA-7908: [DOCS] More clarifications on the

[Impala-ASF-CR] IMPALA-7908: [DOCS] Take 2 More clarifications on the new Admission options

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12028 ) Change subject: IMPALA-7908: [DOCS] Take 2 More clarifications on the new Admission options .. Patch Set 1: Verified+1 Build

[Impala-ASF-CR] IMPALA-7908: [DOCS] Take 2 More clarifications on the new Admission options

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12028 ) Change subject: IMPALA-7908: [DOCS] Take 2 More clarifications on the new Admission options .. Patch Set 2: Verified+1 Build

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 11: Build Successful

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

2018-12-03 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 8: Build Successful

[Impala-ASF-CR] IMPALA-7842: Expose physical plan for unit testing

2018-12-03 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/11920 ) Change subject: IMPALA-7842: Expose physical plan for unit testing .. Patch Set 5: (5 comments) Bharath, thanks much for the review.

[Impala-ASF-CR] IMPALA-7842: Expose physical plan for unit testing

2018-12-03 Thread Paul Rogers (Code Review)
Hello Bharath Vissapragada, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11920 to look at the new patch set (#5). Change subject: IMPALA-7842: Expose physical plan for unit testing

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

2018-12-03 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 9: Build Successful

[Impala-ASF-CR] IMPALA-6994: Avoid reloading a table's HMS data for file-only operations

2018-12-03 Thread Pranay Singh (Code Review)
Pranay Singh has abandoned this change. ( http://gerrit.cloudera.org:8080/10587 ) Change subject: IMPALA-6994: Avoid reloading a table's HMS data for file-only operations .. Abandoned -- To view, visit

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

2018-12-03 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 2: > (1 comment) Thanks Tim, good

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 13: Build Successful

[Impala-ASF-CR] IMPALA-7473: fix crash when printing decimal with -v=3

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12029 ) Change subject: IMPALA-7473: fix crash when printing decimal with -v=3 .. Patch Set 1: Build Successful

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 11: (1 comment)

[Impala-ASF-CR] IMPALA-7865: Repeated type widening of arithmetic expressions

2018-12-03 Thread Paul Rogers (Code Review)
Paul Rogers has abandoned this change. ( http://gerrit.cloudera.org:8080/11952 ) Change subject: IMPALA-7865: Repeated type widening of arithmetic expressions .. Abandoned IMPALA-7902/12001 has a slightly more general

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 11: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 12: (3 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 12: Build started:

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 14: Code-Review+2 (2 comments)

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Tim Armstrong (Code Review)
Hello Pooja Nilangekar, Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10928 to look at the new patch set (#14). Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 15: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-7811: optionally count JVM heap towards process mem limit

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10928 ) Change subject: IMPALA-7811: optionally count JVM heap towards process mem limit .. Patch Set 15: Build started:

[Impala-ASF-CR] IMPALA-7473: fix crash when printing decimal with -v=3

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12029 Change subject: IMPALA-7473: fix crash when printing decimal with -v=3 .. IMPALA-7473: fix crash when printing decimal with -v=3 The

[Impala-ASF-CR] IMPALA-7903: Fix DCHECK failure in RawValue::PrintValue()

2018-12-03 Thread Gabor Kaszab (Code Review)
Hello Csaba Ringhofer, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12021 to look at the new patch set (#2). Change subject: IMPALA-7903: Fix DCHECK failure in RawValue::PrintValue()

[Impala-ASF-CR] IMPALA-7903: Fix DCHECK failure in RawValue::PrintValue()

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12021 ) Change subject: IMPALA-7903: Fix DCHECK failure in RawValue::PrintValue() .. Patch Set 2: Build Successful

[Impala-ASF-CR] IMPALA-7903: Fix DCHECK failure in RawValue::PrintValue()

2018-12-03 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. ( http://gerrit.cloudera.org:8080/12021 ) Change subject: IMPALA-7903: Fix DCHECK failure in RawValue::PrintValue() .. Patch Set 2: (3 comments)

[Impala-ASF-CR] IMPALA-2343: Add lifecycle timeline to plan nodes

2018-12-03 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/11992 ) Change subject: IMPALA-2343: Add lifecycle timeline to plan nodes .. Patch Set 10: Code-Review+1 Thanks for rebasing in a separate PS.

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12000 ) Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-7659: Populate NULL count while computing column stats

2018-12-03 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#7) to the change originally created by piotr.findei...@gmail.com. ( http://gerrit.cloudera.org:8080/11565 ) Change subject: IMPALA-7659: Populate NULL count while computing column stats

[Impala-ASF-CR] IMPALA-7659: Populate NULL count while computing column stats

2018-12-03 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/11565 ) Change subject: IMPALA-7659: Populate NULL count while computing column stats .. Patch Set 7: (1 comment) Rebased. Core and

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 8: (14 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 9: (3 comments)

[Impala-ASF-CR] IMPALA-6955: fix test query concurrency and server startup sequence

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12019 ) Change subject: IMPALA-6955: fix test_query_concurrency and server startup sequence .. Patch Set 1: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 10: Code-Review+1 carry -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#9). Change subject: IMPALA-1048: show sinks in exec summary

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 9: Build Successful

[Impala-ASF-CR] IMPALA-7842: Make query fragments available for unit testing

2018-12-03 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/11920 ) Change subject: IMPALA-7842: Make query fragments available for unit testing .. Patch Set 4: (9 comments) Refactor looks fine

[Impala-ASF-CR] IMPALA-7659: Populate NULL count while computing column stats

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11565 ) Change subject: IMPALA-7659: Populate NULL count while computing column stats .. Patch Set 7: Build Successful

[Impala-ASF-CR] IMPALA-7881 (Part 2): Visualize AST for easier debugging

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12015 ) Change subject: IMPALA-7881 (Part 2): Visualize AST for easier debugging .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 10: Build started:

[Impala-ASF-CR] IMPALA-6955: fix test query concurrency and server startup sequence

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12019 ) Change subject: IMPALA-6955: fix test_query_concurrency and server startup sequence .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7881 (Part 2): Visualize AST for easier debugging

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12015 ) Change subject: IMPALA-7881 (Part 2): Visualize AST for easier debugging .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7881 (Part 2): Visualize AST for easier debugging

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12015 ) Change subject: IMPALA-7881 (Part 2): Visualize AST for easier debugging .. Patch Set 2: Code-Review+2 -- To view, visit

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

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong 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 2: (1 comment)

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

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins 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 1: Build Successful

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12000 ) Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates .. Patch Set 5: Code-Review+2 (1 comment) Had a minor

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

2018-12-03 Thread Sahil Takiar (Code Review)
Hello Lars Volker, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12024 to look at the new patch set (#2). Change subject: IMPALA-7625: test_web_pages.py backend tests are failing

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

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

[Impala-ASF-CR] Build parquet-reader earlier in test-with-docker

2018-12-03 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12025 ) Change subject: Build parquet-reader earlier in test-with-docker .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-2343: Add lifecycle timeline to plan nodes

2018-12-03 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/11992 ) Change subject: IMPALA-2343: Add lifecycle timeline to plan nodes .. Patch Set 10: Code-Review+2 (1 comment)

[Impala-ASF-CR] Build parquet-reader earlier in test-with-docker

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12025 ) Change subject: Build parquet-reader earlier in test-with-docker .. Patch Set 2: Build Successful

[Impala-ASF-CR] IMPALA-7738: Implement timeouts for HDFS open calls

2018-12-03 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/11874 ) Change subject: IMPALA-7738: Implement timeouts for HDFS open calls .. Patch Set 11: Code-Review+2 (2 comments)

[Impala-ASF-CR] [DOCS] Fixed typos in impala math functions examples

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12026 ) Change subject: [DOCS] Fixed typos in impala_math_functions examples .. Patch Set 1: Verified+1 Build Successful

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

2018-12-03 Thread Tianyi Wang (Code Review)
Tianyi Wang 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: (1 comment)

[Impala-ASF-CR] IMPALA-6955: fix test query concurrency and server startup sequence

2018-12-03 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12019 ) Change subject: IMPALA-6955: fix test_query_concurrency and server startup sequence .. Patch Set 1: (1 comment)

[Impala-ASF-CR] [DOCS] Fixed typos in impala math functions examples

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12026 ) Change subject: [DOCS] Fixed typos in impala_math_functions examples .. Patch Set 1: Build Started

[Impala-ASF-CR] Build parquet-reader earlier in test-with-docker

2018-12-03 Thread Laszlo Gaal (Code Review)
Laszlo Gaal has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12025 Change subject: Build parquet-reader earlier in test-with-docker .. Build parquet-reader earlier in test-with-docker Test-with-docker

[Impala-ASF-CR] [DOCS] Fixed typos in impala math functions examples

2018-12-03 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12026 Change subject: [DOCS] Fixed typos in impala_math_functions examples .. [DOCS] Fixed typos in impala_math_functions examples Change-Id:

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12000 ) Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates .. Patch Set 5: (2 comments)

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12000 ) Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates .. Patch Set 6: Code-Review+2 Carry Lars' +2. -- To

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Michael Ho (Code Review)
Hello Lars Volker, Balazs Jeszenszky, Zoram Thanga, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12000 to look at the new patch set (#5). Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12000 ) Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates .. Patch Set 5: (2 comments)

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12000 ) Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-6741: Add timestamp of fragment instance's status updates

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12000 ) Change subject: IMPALA-6741: Add timestamp of fragment instance's status updates .. Patch Set 5: Build Successful

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

2018-12-03 Thread Sahil Takiar (Code Review)
Sahil Takiar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12024 Change subject: IMPALA-7625: test_web_pages.py backend tests are failing .. IMPALA-7625: test_web_pages.py backend tests are failing

[Impala-ASF-CR] IMPALA-6955: fix test query concurrency and server startup sequence

2018-12-03 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12019 ) Change subject: IMPALA-6955: fix test_query_concurrency and server startup sequence .. Patch Set 1: Code-Review+2 (1 comment)

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

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins 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 1: (5 comments)

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

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12001 ) Change subject: IMPALA-7902: NumericLiteral fixes, refactoring .. Patch Set 7: (10 comments) Did a pass over this. Didn't fully

[Impala-ASF-CR](2.x) IMPALA-7922: Fix 2.x broken build due to Jackson dependency update

2018-12-03 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12027 ) Change subject: IMPALA-7922: Fix 2.x broken build due to Jackson dependency update .. Patch Set 3: Code-Review+2 If we happen to

[Impala-ASF-CR](2.x) IMPALA-7922: Fix 2.x broken build due to Jackson dependency update

2018-12-03 Thread Quanlong Huang (Code Review)
Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12027 ) Change subject: IMPALA-7922: Fix 2.x broken build due to Jackson dependency update .. Patch Set 3: Thank you, Fredy! I'm curious

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

2018-12-03 Thread Paul Rogers (Code Review)
Paul Rogers 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: (2 comments)

  1   2   >