[Impala-ASF-CR] IMPALA-5162,IMPALA-5163: stress test support on secure clusters

2017-05-01 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-5162,IMPALA-5163: stress test support on secure clusters .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-5162,IMPALA-5163: stress test support on secure clusters

2017-05-01 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5162,IMPALA-5163: stress test support on secure clusters .. Patch Set 1: Code-Review+2 (2 comments)

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

2017-05-01 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5266 Impala ABM / LZCNT support .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5821/5/be/src/util/bit-util.h File be/src/util/bit-util.h:

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6559 to look at the new patch set (#9). Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables ..

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6559 To

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 9: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/523/ --

[Impala-ASF-CR] IMPALA-5262: test analytic order by random fails with assert

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5262: test_analytic_order_by_random fails with assert .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-5137: pt1, Refactor TimestampValue constructors

2017-05-01 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5137: pt1, Refactor TimestampValue constructors .. Patch Set 7: Code-Review+2 sorry for all the rebasing noise, I'm holding off on committing this until

[Impala-ASF-CR] IMPALA-5162,IMPALA-5163: stress test support on secure clusters

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5162,IMPALA-5163: stress test support on secure clusters .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-5262: test analytic order by random fails with assert

2017-05-01 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5262: test_analytic_order_by_random fails with assert .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6775/1/tests/query_test/test_sort.py

[Impala-ASF-CR] IMPALA-2550: Switch to per-query exec rpc

2017-05-01 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-2550: Switch to per-query exec rpc .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/6535/10/be/src/runtime/descriptors.h File

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 9: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6559 To

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. IMPALA-3742: Partitions and sort INSERTs for Kudu tables Bulk DMLs (INSERT, UPSERT,

[Impala-ASF-CR] IMPALA-5137: Support Kudu UNIXTIME MICROS as Impala TIMESTAMP

2017-05-01 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#5). Change subject: IMPALA-5137: Support Kudu UNIXTIME_MICROS as Impala TIMESTAMP .. IMPALA-5137: Support Kudu UNIXTIME_MICROS as Impala TIMESTAMP Adds Impala support for TIMESTAMP

[Impala-ASF-CR] Experiment: glibc strncmp/memcmp appears much faster than SSE4.2

2017-05-01 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Experiment: glibc strncmp/memcmp appears much faster than SSE4.2 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6768/1//COMMIT_MSG Commit Message:

[Impala-ASF-CR] Bump Kudu version to 238249c

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Bump Kudu version to 238249c .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/520/ -- To view, visit

[Impala-ASF-CR] Experiment: glibc strncmp/memcmp appears much faster than SSE4.2

2017-05-01 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: Experiment: glibc strncmp/memcmp appears much faster than SSE4.2 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6768/1//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-5003: Constant propagation in scan conjuncts

2017-05-01 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5003: Constant propagation in scan conjuncts .. Patch Set 21: Got a green light test run for this:

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

2017-05-01 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: Impala ABM / LZCNT support .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5821/3//COMMIT_MSG Commit Message: Line 7: Impala ABM / LZCNT support

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

2017-05-01 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#5). Change subject: IMPALA-5266 Impala ABM / LZCNT support .. IMPALA-5266 Impala ABM / LZCNT support I recently added some code that wants to do upwards power of 2 calculation. Turns

[Impala-ASF-CR] IMPALA-5162,IMPALA-5163: stress test support on secure clusters

2017-05-01 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-5162,IMPALA-5163: stress test support on secure clusters .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-5162,IMPALA-5163: stress test support on secure clusters

2017-05-01 Thread Matthew Mulder (Code Review)
Matthew Mulder has posted comments on this change. Change subject: IMPALA-5162,IMPALA-5163: stress test support on secure clusters .. Patch Set 1: (1 comment)

[Impala-ASF-CR] Bump Kudu version to 238249c

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: Bump Kudu version to 238249c .. Bump Kudu version to 238249c This will pull in the Kudu client partitioner API, which is needed for

[Impala-ASF-CR] Bump Kudu version to 238249c

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Bump Kudu version to 238249c .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6718 To unsubscribe, visit

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

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet .. Patch Set 10: Build started:

[Impala-ASF-CR] IMPALA-5003: Constant propagation in scan conjuncts

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5003: Constant propagation in scan conjuncts .. Patch Set 25: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/522/ -- To

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6559 to look at the new patch set (#8). Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables ..

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 8: Code-Review+2 (2 comments)

[Impala-ASF-CR] IMPALA-5162,IMPALA-5163: stress test support on secure clusters

2017-05-01 Thread Matthew Mulder (Code Review)
Matthew Mulder has posted comments on this change. Change subject: IMPALA-5162,IMPALA-5163: stress test support on secure clusters .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6763 To

[Impala-ASF-CR] IMPALA-5262: test analytic order by random fails with assert

2017-05-01 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5262: test_analytic_order_by_random fails with assert .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6775/1/tests/query_test/test_sort.py

[Impala-ASF-CR] IMPALA-5162,IMPALA-5163: stress test support on secure clusters

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5162,IMPALA-5163: stress test support on secure clusters .. IMPALA-5162,IMPALA-5163: stress test support on secure clusters This patch

[Impala-ASF-CR] IMPALA-5162,IMPALA-5163: stress test support on secure clusters

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5162,IMPALA-5163: stress test support on secure clusters .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6763 To

[Impala-ASF-CR] IMPALA-5180: Don't use non-deterministic exprs in partition pruning

2017-05-01 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5180: Don't use non-deterministic exprs in partition pruning .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6575 To

[Impala-ASF-CR] IMPALA-5180: Don't use non-deterministic exprs in partition pruning

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5180: Don't use non-deterministic exprs in partition pruning .. Patch Set 9: Build started:

[Impala-ASF-CR] IMPALA-4866: Hash join node does not apply limits correctly

2017-05-01 Thread anujphadke (Code Review)
anujphadke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6778 Change subject: IMPALA-4866: Hash join node does not apply limits correctly .. IMPALA-4866: Hash join node does not apply limits correctly

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

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet .. Patch Set 10: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-5003: Constant propagation in scan conjuncts

2017-05-01 Thread Alex Behm (Code Review)
Alex Behm has submitted this change and it was merged. Change subject: IMPALA-5003: Constant propagation in scan conjuncts .. IMPALA-5003: Constant propagation in scan conjuncts Implements constant propagation within conjuncts

[Impala-ASF-CR] IMPALA-5003: Constant propagation in scan conjuncts

2017-05-01 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5003: Constant propagation in scan conjuncts .. Patch Set 25: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6389 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5262: test analytic order by random fails with assert

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6775 Change subject: IMPALA-5262: test_analytic_order_by_random fails with assert .. IMPALA-5262: test_analytic_order_by_random

[Impala-ASF-CR] IMPALA-5262: test analytic order by random fails with assert

2017-05-01 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5262: test_analytic_order_by_random fails with assert .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6775/1/tests/query_test/test_sort.py

[Impala-ASF-CR] IMPALA-5003: Constant propagation in scan conjuncts

2017-05-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5003: Constant propagation in scan conjuncts .. Patch Set 25: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6389 To unsubscribe,

[Impala-ASF-CR] IMPALA-5162,IMPALA-5163: stress test support on secure clusters

2017-05-01 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5162,IMPALA-5163: stress test support on secure clusters .. Patch Set 1: (2 comments) Changes look reasonable, but I'm not super familiar with this code.