[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-14 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-13 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 1: Sorry, didn't realize this was the upstream

[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-13 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 1: Sounds like

[Impala-ASF-CR] IMPALA-6148: Specifying thirdparty deps as URLs

2017-11-07 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8456 ) Change subject: IMPALA-6148: Specifying thirdparty deps as URLs .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-5607: Add additional units to EXTRACT, DATE PART, TRUNC

2017-10-30 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8311 ) Change subject: IMPALA-5607: Add additional units to EXTRACT, DATE_PART, TRUNC .. Patch Set 4: (3 comments) Let's figure out how to

[Impala-ASF-CR] IMPALA-5607: Add additional units to EXTRACT, DATE PART, TRUNC

2017-10-26 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8311 ) Change subject: IMPALA-5607: Add additional units to EXTRACT, DATE_PART, TRUNC .. Patch Set 2: Looks like we want NANOSECONDS after all,

[Impala-ASF-CR] IMPALA-5607: Add additional units to EXTRACT, DATE PART, TRUNC

2017-10-26 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8311 ) Change subject: IMPALA-5607: Add additional units to EXTRACT, DATE_PART, TRUNC .. Patch Set 2: (6 comments) Overall this looks good -

[Impala-ASF-CR] IMPALA-6068: Fix dataload for complextypes fileformat

2017-10-24 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8350 ) Change subject: IMPALA-6068: Fix dataload for complextypes_fileformat .. Patch Set 5: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

2017-10-20 Thread Zach Amsden (Code Review)
Zach Amsden has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8274 ) Change subject: IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+ .. IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

[Impala-ASF-CR] IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

2017-10-19 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8274 ) Change subject: IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+ .. Patch Set 4: Looks like I finally got a good GVO on this. Tim,

[Impala-ASF-CR] IMPALA-6070: Parallel data load.

2017-10-18 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8320 ) Change subject: IMPALA-6070: Parallel data load. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

2017-10-18 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8274 ) Change subject: IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+ .. Patch Set 4: Anyone know what's going on with this change not

[Impala-ASF-CR] IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

2017-10-18 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8274 ) Change subject: IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+ .. Patch Set 2: Looks like a transient failure, retrying -- To

[Impala-ASF-CR] IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

2017-10-13 Thread Zach Amsden (Code Review)
Hello Philip Zeyliger, Joe McDonnell, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8274 to look at the new patch set (#2). Change subject: IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

[Impala-ASF-CR] IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

2017-10-13 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8274 ) Change subject: IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+ .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+

2017-10-13 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8274 Change subject: IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+ .. IMPALA-6055: Fix hdfs encryption test far Hadoop 2.8+ Hadoop

[Impala-ASF-CR] IMPALA-4939, IMPALA-4940: Decimal V2 multiplication

2017-09-28 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/7438 ) Change subject: IMPALA-4939, IMPALA-4940: Decimal V2 multiplication .. Patch Set 6: The algorithm mentioned here, on page 17 seems quite

[Impala-ASF-CR] IMPALA-4939, IMPALA-4940: Decimal V2 multiplication

2017-09-26 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/7438 ) Change subject: IMPALA-4939, IMPALA-4940: Decimal V2 multiplication .. Patch Set 6: I totally agree about doing anything fancy as a

[Impala-ASF-CR] IMPALA-5975: Work around broken beeline clients

2017-09-26 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8132 ) Change subject: IMPALA-5975: Work around broken beeline clients .. Patch Set 1: >From private build and test 6418, 14:21:05 Executing:

[Impala-ASF-CR] IMPALA-5975: Work around broken beeline clients

2017-09-25 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8132 ) Change subject: IMPALA-5975: Work around broken beeline clients .. Patch Set 1: I tested this with a local data load, but we need to

[Impala-ASF-CR] IMPALA-5975: Work around broken beeline clients

2017-09-22 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8132 ) Change subject: IMPALA-5975: Work around broken beeline clients .. Patch Set 1: I will do that now. -- To view, visit

[Impala-ASF-CR] IMPALA-5975: Work around broken beeline clients

2017-09-22 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8132 Change subject: IMPALA-5975: Work around broken beeline clients .. IMPALA-5975: Work around broken beeline clients To make statements

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-22 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8004 ) Change subject: IMPALA-4513: Promote integer types for ABS() .. Patch Set 7: Finally got back to this, looks like this was the only

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-22 Thread Zach Amsden (Code Review)
Hello Lars Volker, Michael Brown, Tim Armstrong, Alex Behm, Dan Hecht, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8004 to look at the new patch set (#6). Change subject: IMPALA-4513: Promote integer types for ABS()

[Impala-ASF-CR] IMPALA-4939, IMPALA-4940: Decimal V2 multiplication

2017-09-20 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4939, IMPALA-4940: Decimal V2 multiplication .. Patch Set 5: Have we ever tried using SSE/AVX for multiplication? It should be possible to avoid using

[Impala-ASF-CR] IMPALA-5920: Remove admission control dependency on YARN RM jar

2017-09-18 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5920: Remove admission control dependency on YARN RM jar .. Patch Set 5: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-5920: Remove admission control dependency on YARN RM jar

2017-09-18 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5920: Remove admission control dependency on YARN RM jar .. Patch Set 4: (8 comments)

[Impala-ASF-CR] IMPALA-5920: Remove admission control dependency on YARN RM jar

2017-09-18 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5920: Remove admission control dependency on YARN RM jar .. Patch Set 4: (17 comments) Looks good but I have a few comments - there is a lot more that

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-15 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4513: Promote integer types for ABS() .. Patch Set 5: Sorry, been under C6 crunch. Will get back to this today. -- To view, visit

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-08 Thread Zach Amsden (Code Review)
Hello Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8004 to look at the new patch set (#4). Change subject: IMPALA-4513: Promote integer types for ABS() .. IMPALA-4513:

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-08 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4513: Promote integer types for ABS() .. Patch Set 3: (2 comments) Decimal is immune to this because the range is clamped. Float and double are immune

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-07 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#3). Change subject: IMPALA-4513: Promote integer types for ABS() .. IMPALA-4513: Promote integer types for ABS() The internal representation of the most negative number in two's

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-07 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4513: Promote integer types for ABS() .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/8004/2/be/src/exprs/expr-test.cc File

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-07 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4513: Promote integer types for ABS() .. Patch Set 2: As per Greg's request, add overflow check for BIGINT. All other types just get promoted. -- To

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-07 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#2). Change subject: IMPALA-4513: Promote integer types for ABS() .. IMPALA-4513: Promote integer types for ABS() The internal representation of the most negative number in 2's

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-07 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4513: Promote integer types for ABS() .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8004/1/be/src/exprs/expr-test.cc File

[Impala-ASF-CR] IMPALA-4513: Promote integer types for ABS()

2017-09-07 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/8004 Change subject: IMPALA-4513: Promote integer types for ABS() .. IMPALA-4513: Promote integer types for ABS() The internal representation

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-09-05 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: Need to get back to this now that I have smokes fixed. I imagine I'll want to do things a

[Impala-ASF-CR] IMPALA-5854: Update external hadoop versions

2017-08-30 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5854: Update external hadoop versions .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7892/1/bin/impala-config.sh File

[Impala-ASF-CR] IMPALA-5854: Update external hadoop versions

2017-08-30 Thread Zach Amsden (Code Review)
Hello Michael Brown, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7892 to look at the new patch set (#2). Change subject: IMPALA-5854: Update external hadoop versions .. IMPALA-5854:

[Impala-ASF-CR] IMPALA-5854: Update external hadoop versions

2017-08-29 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5854: Update external hadoop versions .. Patch Set 1: Components are already built, S3 buckets updated, and I ran a private exhaustive build & load job,

[Impala-ASF-CR] IMPALA-5854: Update external hadoop versions

2017-08-29 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/7892 Change subject: IMPALA-5854: Update external hadoop versions .. IMPALA-5854: Update external hadoop versions These versions need to be

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-15 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7581/2/bin/impala-config.sh File

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-14 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: (4 comments) A couple of minor nits. Also, let me make sure I didn't accidentally introduce

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-10 Thread Zach Amsden (Code Review)
Zach Amsden has abandoned this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Abandoned Gerrit created another change for some reason. -- To view, visit http://gerrit.cloudera.org:8080/7648 To

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-10 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#2). Change subject: IMPALA-5764: Allow overriding packaged components .. IMPALA-5764: Allow overriding packaged components For allowing multiple different distributions to build

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-10 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/7648 Change subject: IMPALA-5764: Allow overriding packaged components .. IMPALA-5764: Allow overriding packaged components For allowing

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-10 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: needs rebase against merged changes, coming presently -- To view, visit

[Impala-ASF-CR] IMPALA-5768: Better developer documentation

2017-08-07 Thread Zach Amsden (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7350 to look at the new patch set (#5). Change subject: IMPALA-5768: Better developer documentation .. IMPALA-5768:

[Impala-ASF-CR] IMPALA-5768: Better developer documentation

2017-08-04 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5768: Better developer documentation .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7350/2/bin/impala-config.sh File

[Impala-ASF-CR] IMPALA-5768: Better developer documentation

2017-08-04 Thread Zach Amsden (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7350 to look at the new patch set (#4). Change subject: IMPALA-5768: Better developer documentation .. IMPALA-5768:

[Impala-ASF-CR] IMPALA-5768

2017-08-04 Thread Zach Amsden (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7350 to look at the new patch set (#3). Change subject: IMPALA-5768 .. IMPALA-5768 Guide to important environment

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-04 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/7581/1//COMMIT_MSG Commit Message: Line 9:

[Impala-ASF-CR] Guide to important environment variables for build, test, and mini-cluster operations.

2017-08-04 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: Guide to important environment variables for build, test, and mini-cluster operations. .. Patch Set 2: (5 comments)

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-03 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/7581 Change subject: IMPALA-5764: Allow overriding packaged components .. IMPALA-5764: Allow overriding packaged components For allowing

[Impala-ASF-CR] Guide to important environment variables for build, test, and mini-cluster operations.

2017-08-03 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: Guide to important environment variables for build, test, and mini-cluster operations. .. Patch Set 2: I cut this back in scope somewhat and made it use

[Impala-ASF-CR] Guide to important environment variables for build, test, and mini-cluster operations.

2017-08-03 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#2). Change subject: Guide to important environment variables for build, test, and mini-cluster operations. .. Guide to important environment variables for build, test, and mini-cluster

[Impala-ASF-CR] IMPALA-4939, IMPALA-4939: Decimal V2 multiplication

2017-07-26 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/7438/1/be/src/exprs/expr-test.cc File

[Impala-ASF-CR] IMPALA-4939, IMPALA-4939: Decimal V2 multiplication

2017-07-17 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication .. Patch Set 1: (11 comments) http://gerrit.cloudera.org:8080/#/c/7438/1/be/src/exprs/expr-test.cc File

[Impala-ASF-CR] IMPALA-5164: Better benchmark heuristic

2017-07-10 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5164: Better benchmark heuristic .. Patch Set 1: -Code-Review -- To view, visit http://gerrit.cloudera.org:8080/7389 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5164: Better benchmark heuristic

2017-07-10 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/7389 Change subject: IMPALA-5164: Better benchmark heuristic .. IMPALA-5164: Better benchmark heuristic If any benchmark took longer than 50ms

[Impala-ASF-CR] IMPALA-5164: Better benchmark heuristic

2017-07-10 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5164: Better benchmark heuristic .. Patch Set 1: Code-Review+1 Turns out this approach works even better and avoids any special changes to benchmarks. --

[Impala-ASF-CR] IMPALA-5164: Bloom filter benchmark fails to complete

2017-07-10 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5164: Bloom filter benchmark fails to complete .. Patch Set 1: @jbapple, let's go ahead with this diff and if there are more failures, I'll attach more

[Impala-ASF-CR] IMPALA-5164: Bloom filter benchmark fails to complete

2017-07-10 Thread Zach Amsden (Code Review)
Hello Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7381 to look at the new patch set (#2). Change subject: IMPALA-5164: Bloom filter benchmark fails to complete ..

[Impala-ASF-CR] IMPALA-XXXX: Bloom filter benchmark fails to complete

2017-07-07 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-: Bloom filter benchmark fails to complete .. Patch Set 1: @jbapple, Is there a JIRA open for this already? -- To view, visit

[Impala-ASF-CR] IMPALA-5623: Fix lag() on STRING cols to release UDF mem

2017-07-06 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5623: Fix lag() on STRING cols to release UDF mem .. Patch Set 1: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-5498: Support for partial sorts

2017-07-06 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5498: Support for partial sorts .. Patch Set 3: (6 comments) Looks promising! http://gerrit.cloudera.org:8080/#/c/7267/3/be/src/exec/partial-sort-node.cc

[Impala-ASF-CR] Complete guide to important environment variables for build, test, and mini-cluster operations.

2017-07-05 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: Complete guide to important environment variables for build, test, and mini-cluster operations. .. Patch Set 1: (2 comments) > > I personally abhor Google docs

[Impala-ASF-CR] Complete guide to important environment variables for build, test, and mini-cluster operations.

2017-06-30 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: Complete guide to important environment variables for build, test, and mini-cluster operations. .. Patch Set 1: I personally abhor Google docs as they just seem

[Impala-ASF-CR] Complete guide to important environment variables for build, test, and mini-cluster operations.

2017-06-30 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/7350 Change subject: Complete guide to important environment variables for build, test, and mini-cluster operations. .. Complete guide to

[Impala-ASF-CR] IMPALA-5547: Rework FK/PK join detection.

2017-06-29 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5547: Rework FK/PK join detection. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-5128: BE Test Infrastructure

2017-06-27 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5128: BE Test Infrastructure .. Patch Set 2: Code-Review-1 I think we have more important things to do and there are probably even more egregious tests.

[Impala-ASF-CR] IMPALA-5547: Rework FK/PK join detection.

2017-06-21 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5547: Rework FK/PK join detection. .. Patch Set 1: (3 comments)

[Impala-ASF-CR] IMPALA-5548 Fix some minor nits with HDFS parquet column readers

2017-06-21 Thread Zach Amsden (Code Review)
Hello Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7240 to look at the new patch set (#3). Change subject: IMPALA-5548 Fix some minor nits with HDFS parquet column readers

[Impala-ASF-CR] IMPALA-5548 Fix some minor issues with HDFS / parquet column readers

2017-06-21 Thread Zach Amsden (Code Review)
Hello Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7240 to look at the new patch set (#2). Change subject: IMPALA-5548 Fix some minor issues with HDFS / parquet column readers

[Impala-ASF-CR] IMPALA-5548 Fix some minor issues with HDFS / parquet column readers

2017-06-21 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5548 Fix some minor issues with HDFS / parquet column readers .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-06-20 Thread Zach Amsden (Code Review)
Zach Amsden has abandoned this change. Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. Abandoned Not enough win for the complexity. -- To view, visit http://gerrit.cloudera.org:8080/6726 To

[Impala-ASF-CR] IMPALA-5548 Fix some minor issues with HDFS / parquet column readers

2017-06-20 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5548 Fix some minor issues with HDFS / parquet column readers .. Patch Set 1: Some pretty simple fixes that came from work on IMPALA-4864. Running build

[Impala-ASF-CR] IMPALA-5548 Fix some minor issues with HDFS / parquet column readers

2017-06-20 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/7240 Change subject: IMPALA-5548 Fix some minor issues with HDFS / parquet column readers .. IMPALA-5548 Fix some minor issues with HDFS /

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-06-16 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#19). Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. IMPALA-4864 Speed up single slot predicates with dictionaries When dictionaries are present we can

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-06-16 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#18). Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. IMPALA-4864 Speed up single slot predicates with dictionaries When dictionaries are present we can

[Impala-ASF-CR] IMPALA-5036: Parquet count star optimization

2017-06-13 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5036: Parquet count star optimization .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/6812/3/be/src/exec/hdfs-parquet-scanner.cc File

[Impala-ASF-CR] IMPALA-5467: disable flaky BenchmarkTest to unblock builds

2017-06-09 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5467: disable flaky BenchmarkTest to unblock builds .. Patch Set 1: Code-Review+1 Well since the fix, we know this isn't being caused by contention, any

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-06-08 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. Patch Set 16: (1 comment)

[Impala-ASF-CR] IMPALA-5164: Fix flaky benchmarks

2017-06-06 Thread Zach Amsden (Code Review)
Hello Impala Public Jenkins, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6935 to look at the new patch set (#8). Change subject: IMPALA-5164: Fix flaky benchmarks ..

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-06-05 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. Patch Set 16: (5 comments)

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-06-05 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#16). Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. IMPALA-4864 Speed up single slot predicates with dictionaries When dictionaries are present we can

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-06-01 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. Patch Set 14: I'm getting kind of disheartened by this approach. The biggest obstacle is that

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-06-01 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#14). Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. IMPALA-4864 Speed up single slot predicates with dictionaries When dictionaries are present we can

[Impala-ASF-CR] IMPALA-4988: Add query option read parquet statistics

2017-06-01 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4988: Add query option read_parquet_statistics .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-5164: Fix flaky benchmarks

2017-05-31 Thread Zach Amsden (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6935 to look at the new patch set (#6). Change subject: IMPALA-5164: Fix flaky benchmarks .. IMPALA-5164: Fix flaky

[Impala-ASF-CR] IMPALA-5164: Fix flaky benchmarks

2017-05-31 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5164: Fix flaky benchmarks .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6935/5/be/src/benchmarks/free-lists-benchmark.cc File

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-05-31 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#13). Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. IMPALA-4864 Speed up single slot predicates with dictionaries When dictionaries are present we can

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-05-31 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#12). Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. IMPALA-4864 Speed up single slot predicates with dictionaries When dictionaries are present we can

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-05-30 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. Patch Set 9: (13 comments) http://gerrit.cloudera.org:8080/#/c/6726/9/be/src/exec/exec-node.cc

[Impala-ASF-CR] IMPALA-5164: Fix flaky benchmarks

2017-05-30 Thread Zach Amsden (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6935 to look at the new patch set (#5). Change subject: IMPALA-5164: Fix flaky benchmarks .. IMPALA-5164: Fix flaky

[Impala-ASF-CR] IMPALA-5164: Fix flaky benchmarks

2017-05-30 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5164: Fix flaky benchmarks .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6935/4/be/src/util/benchmark.h File be/src/util/benchmark.h:

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-05-26 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#11). Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. IMPALA-4864 Speed up single slot predicates with dictionaries When dictionaries are present we can

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-05-26 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. Patch Set 10: (3 comments)

[Impala-ASF-CR] IMPALA-4864 Speed up single slot predicates with dictionaries

2017-05-26 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#10). Change subject: IMPALA-4864 Speed up single slot predicates with dictionaries .. IMPALA-4864 Speed up single slot predicates with dictionaries When dictionaries are present we can

  1   2   3   4   >