[Impala-ASF-CR] IMPALA-4839: Remove implicit 'localhost' for KUDU MASTER HOSTS

2017-02-08 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-4839: Remove implicit 'localhost' for KUDU_MASTER_HOSTS .. Patch Set 8: Pre-review build: http://jenkins.impala.io:8080/view/Utility/job/pre-review-test/18

[Impala-ASF-CR] IMPALA-2020: Add rounding when casting from decimal to int

2017-02-08 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-2020: Add rounding when casting from decimal to int .. Patch Set 3: > I'm not opposed to cleaning up the AnyVal stuff like that, but > given that udf.h stu

[Impala-ASF-CR] IMPALA-2020: Add rounding when casting from decimal to int

2017-02-08 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#3). Change subject: IMPALA-2020: Add rounding when casting from decimal to int .. IMPALA-2020: Add rounding when casting from decimal to int Preview diff, not working yet, mostly to see

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#11). Change subject: IMPALA-4822: Implement dynamic log level changes .. IMPALA-4822: Implement dynamic log level changes Very often we have to change the logging levels of Imp

[Impala-ASF-CR] IMPALA-4810: Make DECIMAL expr-test cases table driven

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4810: Make DECIMAL expr-test cases table driven .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5933/4/be/src/testutil/impalad-query-executor

[Impala-ASF-CR] IMPALA-4818: Ensure the same number of tests are run every time

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4818: Ensure the same number of tests are run every time .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5784 To unsubscribe,

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 20: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/253/ -- To view, visi

[Impala-ASF-CR] IMPALA-2020: Add rounding when casting from decimal to int

2017-02-08 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-2020: Add rounding when casting from decimal to int .. Patch Set 2: I'm not opposed to cleaning up the AnyVal stuff like that, but given that udf.h stuff dic

[Impala-ASF-CR] IMPALA-4884: Add JVM heap and non-heap usage in metrics and UI

2017-02-08 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4884: Add JVM heap and non-heap usage in metrics and UI .. Patch Set 2: Code-Review+1 Keep Alex's +1 -- To view, visit http://gerrit.cloudera.or

[Impala-ASF-CR] IMPALA-4884: Add JVM heap and non-heap usage in metrics and UI

2017-02-08 Thread Dimitris Tsirogiannis (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5909 to look at the new patch set (#2). Change subject: IMPALA-4884: Add JVM heap and non-heap usage in metrics and UI .. I

[Impala-ASF-CR] IMPALA-4884: Add JVM heap and non-heap usage in metrics and UI

2017-02-08 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4884: Add JVM heap and non-heap usage in metrics and UI .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5909/1/be/src/util/defau

[Impala-ASF-CR] IMPALA-2020: Add rounding when casting from decimal to int

2017-02-08 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#2). Change subject: IMPALA-2020: Add rounding when casting from decimal to int .. IMPALA-2020: Add rounding when casting from decimal to int Preview diff, not working yet, mostly to see

[Impala-ASF-CR] IMPALA-4842: BufferedBlockMgrTest.WriteError is flaky

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4842: BufferedBlockMgrTest.WriteError is flaky .. Patch Set 1: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/251

[Impala-ASF-CR] IMPALA-2020: Add rounding when casting from decimal to int

2017-02-08 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/5951 Change subject: IMPALA-2020: Add rounding when casting from decimal to int .. IMPALA-2020: Add rounding when casting from decimal to int P

[Impala-ASF-CR] IMPALA-4809: Codegen support for DECIMAL V2

2017-02-08 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new change for review. http://gerrit.cloudera.org:8080/5950 Change subject: IMPALA-4809: Codegen support for DECIMAL_V2 .. IMPALA-4809: Codegen support for DECIMAL_V2 Currently, codegen supports conv

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 22: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun

[Impala-ASF-CR] Impala ABM / LZCNT support

2017-02-08 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#3). Change subject: Impala ABM / LZCNT support .. Impala ABM / LZCNT support I recently added some code that wants to do upwards power of 2 calculation. Turns out this can be done much

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4822: Implement dynamic log level changes .. Patch Set 10: PS10 gets rid of TResetJavaLogLevelParams. -- To view, visit http://gerrit.cloudera.or

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#10). Change subject: IMPALA-4822: Implement dynamic log level changes .. IMPALA-4822: Implement dynamic log level changes Very often we have to change the logging levels of Imp

[Impala-ASF-CR] Bump Kudu version to latest Kudu master

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Bump Kudu version to latest Kudu master .. Patch Set 1: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/249/ -- To view

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 20: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/253/ -- To view, visit http://g

[Impala-ASF-CR] IMPALA-4731: Crash when sorting on non-deterministic expr

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4731: Crash when sorting on non-deterministic expr .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/252/ --

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 20: Trivial test fix +2 -- To view, visit http://gerrit.cloudera.org:8080/5776 To unsubscribe, visit http://gerrit.cl

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 20: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5776 To unsubscribe, visit http://gerrit.cloudera.

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. IMPALA-4895: Memory limit exceeded in test_outer_joins A recent change (IMPALA-3524) r

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5941 To unsubscribe,

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-08 Thread Zach Amsden (Code Review)
Hello Impala Public Jenkins, Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5776 to look at the new patch set (#20). Change subject: IMPALA-4729: Implement REPLACE() ...

Re: [Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-02-08 Thread Alex Behm
Patch has some test failures, please fix. On Tue, Feb 7, 2017 at 9:23 PM, Impala Public Jenkins (Code Review) < ger...@cloudera.org> wrote: > Impala Public Jenkins has posted comments on this change. > > Change subject: IMPALA-4729: Implement REPLACE() > ..

[Impala-ASF-CR] IMPALA-4731: Crash when sorting on non-deterministic expr

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4731: Crash when sorting on non-deterministic expr .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5914/2/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-4839: Remove implicit 'localhost' for KUDU MASTER HOSTS

2017-02-08 Thread David Knupp (Code Review)
Hello Michael Brown, Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5877 to look at the new patch set (#8). Change subject: IMPALA-4839: Remove implicit 'localhost' for KUDU_MASTER_HOSTS .

[Impala-ASF-CR] IMPALA-4839: Remove implicit 'localhost' for KUDU MASTER HOSTS

2017-02-08 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-4839: Remove implicit 'localhost' for KUDU_MASTER_HOSTS .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5877/7/tests/conftest.py File tests

[Impala-ASF-CR] IMPALA-4842: BufferedBlockMgrTest.WriteError is flaky

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4842: BufferedBlockMgrTest.WriteError is flaky .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/251/ -- To

[Impala-ASF-CR] IMPALA-4842: BufferedBlockMgrTest.WriteError is flaky

2017-02-08 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4842: BufferedBlockMgrTest.WriteError is flaky .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5940 To unsubscribe, visit htt

[Impala-ASF-CR] IMPALA-4849: IllegalStateException from rewritten CASE expr

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4849: IllegalStateException from rewritten CASE expr .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5917 To unsubscribe, vis

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 22: Code-Review+2 carry +2 -- To view, visit http://gerrit.cloudera.org:8080/5161 To unsu

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 22: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/250/ --

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 20: (2 comments) http://gerrit.cloudera.org:8080/#/c/5161/20/fe/src/main/java/org/apache/i

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5161 to look at the new patch set (#21). Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs ...

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 20: (2 comments) FE lgtm +2 http://gerrit.cloudera.org:8080/#/c/5161/20/fe/src/main/java/org/

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala security topics to be generic

2017-02-08 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala security topics to be generic .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5931 To unsubscribe,

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 19: (1 comment) http://gerrit.cloudera.org:8080/#/c/5161/19/fe/src/main/java/org/apache/im

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5161 to look at the new patch set (#20). Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs ...

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4822: Implement dynamic log level changes .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5792/7/common/thrift/Logging.thrift File common/thr

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4822: Implement dynamic log level changes .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5792/7/common/thrift/Logging.thrift File commo

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4822: Implement dynamic log level changes .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5792/7/common/thrift/Logging.thrift File

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4822: Implement dynamic log level changes .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5792/7/common/thrift/Logging.thrift File common/thr

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4822: Implement dynamic log level changes .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5792/7/common/thrift/Logging.thrift File

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala security topics to be generic

2017-02-08 Thread Ambreen Kazi (Code Review)
Ambreen Kazi has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala security topics to be generic .. Patch Set 1: (14 comments) http://gerrit.cloudera.org:8080/#/c/5931/1//COMMIT_MSG Commit Messag

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala security topics to be generic

2017-02-08 Thread Ambreen Kazi (Code Review)
Ambreen Kazi has uploaded a new patch set (#3). Change subject: IMPALA-3410 [DOCS] Rework Impala security topics to be generic .. IMPALA-3410 [DOCS] Rework Impala security topics to be generic This is part 1 of the changes being

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 19: (1 comment) http://gerrit.cloudera.org:8080/#/c/5161/19/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-4643: [DOCS] Set up many new keydefs

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4643: [DOCS] Set up many new keydefs .. IMPALA-4643: [DOCS] Set up many new keydefs Make keydefs corresponding to most of the links (espe

[Impala-ASF-CR] IMPALA-4643: [DOCS] Set up many new keydefs

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4643: [DOCS] Set up many new keydefs .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/37/ -- To view, visit h

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 19: (1 comment) http://gerrit.cloudera.org:8080/#/c/5161/19/fe/src/main/java/org/apache/im

[Impala-ASF-CR] IMPALA-4643: [DOCS] Set up many new keydefs

2017-02-08 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: IMPALA-4643: [DOCS] Set up many new keydefs .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5923 To unsubscribe, visit http://ger

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4822: Implement dynamic log level changes .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5792/7/common/thrift/Logging.thrift File common/thr

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala security topics to be generic

2017-02-08 Thread Ambreen Kazi (Code Review)
Ambreen Kazi has uploaded a new patch set (#2). Change subject: IMPALA-3410 [DOCS] Rework Impala security topics to be generic .. IMPALA-3410 [DOCS] Rework Impala security topics to be generic This is part 1 of the changes being

[Impala-ASF-CR] Bump Kudu version to latest Kudu master

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Bump Kudu version to latest Kudu master .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/249/ -- To view, visit ht

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 19: (1 comment) http://gerrit.cloudera.org:8080/#/c/5161/19/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet

2017-02-08 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new patch set (#2). Change subject: IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet .. IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet Before this change

[Impala-ASF-CR] IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet

2017-02-08 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new patch set (#2). Change subject: IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet .. IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet Before this change

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5941 To unsubscribe, visit ht

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/248/ -- To

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5941/1/testdata/workloads/tpch/q

[Impala-ASF-CR] Bump Kudu version to latest Kudu master

2017-02-08 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: Bump Kudu version to latest Kudu master .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5929 To unsubscribe, visit http://gerrit

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Thomas Tauber-Marshall (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5941 to look at the new patch set (#2). Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. IMPALA-48

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5941/1/testdata/workloads/tpch/queries/tp

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. Patch Set 1: Code-Review+1 Seems okay to me. Michael, are you okay with having this "none" mechanism? --

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/5941 Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. IMPALA-4895: Memory limit exceeded in test_outer_joins

[Impala-ASF-CR] IMPALA-4842: BufferedBlockMgrTest.WriteError is flaky

2017-02-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/5940 Change subject: IMPALA-4842: BufferedBlockMgrTest.WriteError is flaky .. IMPALA-4842: BufferedBlockMgrTest.WriteError is flaky The test

[Impala-ASF-CR] IMPALA-4828: Alter Kudu schema outside Impala may crash on read

2017-02-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#5). Change subject: IMPALA-4828: Alter Kudu schema outside Impala may crash on read .. IMPALA-4828: Alter Kudu schema outside Impala may crash on read Creating a table in Impala, cha

[Impala-ASF-CR] IMPALA-4828: Alter Kudu schema outside Impala may crash on read

2017-02-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#4). Change subject: IMPALA-4828: Alter Kudu schema outside Impala may crash on read .. IMPALA-4828: Alter Kudu schema outside Impala may crash on read Creating a table in Impala, cha

[Impala-ASF-CR] IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet

2017-02-08 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new change for review. http://gerrit.cloudera.org:8080/5939 Change subject: IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet .. IMPALA-2716: Hive/Impala incompatibility for ti

[Impala-ASF-CR] IMPALA-4828: Alter Kudu schema outside Impala may crash on read

2017-02-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4828: Alter Kudu schema outside Impala may crash on read .. Patch Set 2: After discussing with Dan on the Kudu team, we can make this simpler by checkin

[Impala-ASF-CR] IMPALA-3748: Part 1: Clean up resource estimation in planner

2017-02-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3748: Part 1: Clean up resource estimation in planner .. Patch Set 4: (5 comments) Some minor cleanup and tests fixes http://gerrit.cloudera.org:8080/#/

[Impala-ASF-CR] IMPALA-3748: Part 1: Clean up resource estimation in planner

2017-02-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). Change subject: IMPALA-3748: Part 1: Clean up resource estimation in planner .. IMPALA-3748: Part 1: Clean up resource estimation in planner This is in preparation to use this cod

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5161 to look at the new patch set (#19). Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs ...

[Impala-ASF-CR] IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs

2017-02-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-1430,IMPALA-4878,IMPALA-4879: codegen native UDAs .. Patch Set 19: (2 comments) http://gerrit.cloudera.org:8080/#/c/5161/18/fe/src/main/java/org/apache/i

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#9). Change subject: IMPALA-4822: Implement dynamic log level changes .. IMPALA-4822: Implement dynamic log level changes Very often we have to change the logging levels of Impa

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#8). Change subject: IMPALA-4822: Implement dynamic log level changes .. IMPALA-4822: Implement dynamic log level changes Very often we have to change the logging levels of Impa

[Impala-ASF-CR] IMPALA-4822: Implement dynamic log level changes

2017-02-08 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4822: Implement dynamic log level changes .. Patch Set 7: (29 comments) http://gerrit.cloudera.org:8080/#/c/5792/7/be/src/service/impala-http-hand