[Impala-ASF-CR] IMPALA-3437: DECIMAL V2: avoid implicit decimal->double conversion

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3437: DECIMAL_V2: avoid implicit decimal->double conversion .. Patch Set 6: (1 comment)

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Update download and signature links for 2.10.0 release. .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/8052 To unsubscribe,

[Impala-ASF-CR] IMPALA-5668: Fix cast(X as timestamp) for negative subsecond Decimals

2017-09-13 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-5668: Fix cast(X as timestamp) for negative subsecond Decimals .. Patch Set 1: As discussed via email, let's add tests after IMPALA-5664 has been fixed.

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new change for review. http://gerrit.cloudera.org:8080/8052 Change subject: Update download and signature links for 2.10.0 release. .. Update download and signature links for 2.10.0

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Update download and signature links for 2.10.0 release. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8052/1/downloads.html File downloads.html:

[Impala-ASF-CR] IMPALA-3437: DECIMAL V2: avoid implicit decimal->double conversion

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3437: DECIMAL_V2: avoid implicit decimal->double conversion .. Patch Set 5: (10 comments)

[Impala-ASF-CR] IMPALA-3437: DECIMAL V2: avoid implicit decimal->double conversion

2017-09-13 Thread Tim Armstrong (Code Review)
Hello Greg Rahn, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7916 to look at the new patch set (#6). Change subject: IMPALA-3437: DECIMAL_V2: avoid implicit decimal->double conversion

[Impala-ASF-CR] IMPALA-5905: add script for all-build-options job

2017-09-13 Thread Tim Armstrong (Code Review)
Hello Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8043 to look at the new patch set (#3). Change subject: IMPALA-5905: add script for all-build-options job .. IMPALA-5905:

[Impala-ASF-CR] IMPALA-5905: add script for all-build-options job

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5905: add script for all-build-options job .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/8043/1/bin/all-build-options.sh File

[Impala-ASF-CR] IMPALA-5905: add script for all-build-options job

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5905: add script for all-build-options job .. Patch Set 3: Code-Review+2 Verified+1 Carry +2. Verified manually by running locally (no point in running

[Impala-ASF-CR] IMPALA-5905: add script for all-build-options job

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. Change subject: IMPALA-5905: add script for all-build-options job .. IMPALA-5905: add script for all-build-options job This checks in a modified version of the job

[Impala-ASF-CR] IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

2017-09-13 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 7: (5 comments)

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: Update download and signature links for 2.10.0 release. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8052/2/downloads.html File

[Impala-ASF-CR] IMPALA-1649 Pass precision to Print TCounterType::DOUBLE VALUE

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE .. IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE Fix: Current code uses

[Impala-ASF-CR](asf-site) Revert "Update download and signature links for 2.10.0 release."

2017-09-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new change for review. http://gerrit.cloudera.org:8080/8057 Change subject: Revert "Update download and signature links for 2.10.0 release." .. Revert "Update download and signature links

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8050/1/be/src/service/child-query.cc

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/8056 Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: Update download and signature links for 2.10.0 release. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8052/1/downloads.html File

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Update download and signature links for 2.10.0 release. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8052/2/downloads.html File downloads.html:

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Bharath Vissapragada (Code Review)
Hello Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8052 to look at the new patch set (#3). Change subject: Update download and signature links for 2.10.0 release. ..

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: Update download and signature links for 2.10.0 release. .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8052 To

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has submitted this change and it was merged. Change subject: Update download and signature links for 2.10.0 release. .. Update download and signature links for 2.10.0 release. Change-Id:

[Impala-ASF-CR](asf-site) Update download and signature links for 2.10.0 release.

2017-09-13 Thread Bharath Vissapragada (Code Review)
Hello Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8052 to look at the new patch set (#2). Change subject: Update download and signature links for 2.10.0 release. ..

[Impala-ASF-CR](asf-site) Revert "Update download and signature links for 2.10.0 release."

2017-09-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: Revert "Update download and signature links for 2.10.0 release." .. Patch Set 1: ASF policy requires that we need to wait for at least 24 hours before

[Impala-ASF-CR] IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

2017-09-13 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has uploaded a new patch set (#8). Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. IMPALA-1767 Adds predicate to test boolean values true, false, unknown. Adds a new

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Lars Volker (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8050 to look at the new patch set (#3). Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() ..

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/8050/3//COMMIT_MSG Commit Message:

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

2017-09-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5920: Remove admission control dependency on YARN RM jar .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/8035/2//COMMIT_MSG Commit

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

2017-09-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#3). Change subject: IMPALA-5920: Remove admission control dependency on YARN RM jar .. IMPALA-5920: Remove admission control dependency on YARN RM jar Impala's admission controller

[Impala-ASF-CR] IMPALA-5250: Unify decompressor output length semantics

2017-09-13 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. Change subject: IMPALA-5250: Unify decompressor output_length semantics .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8030/1//COMMIT_MSG Commit Message: Line

[Impala-ASF-CR](asf-site) Revert "Update download and signature links for 2.10.0 release."

2017-09-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has submitted this change and it was merged. Change subject: Revert "Update download and signature links for 2.10.0 release." .. Revert "Update download and signature links for 2.10.0 release." This

[Impala-ASF-CR](asf-site) Revert "Update download and signature links for 2.10.0 release."

2017-09-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: Revert "Update download and signature links for 2.10.0 release." .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8057 To

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 3: Code-Review+2 Carrying Alex's +2. -- To view, visit

[Impala-ASF-CR](asf-site) Revert "Update download and signature links for 2.10.0 release."

2017-09-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Revert "Update download and signature links for 2.10.0 release." .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8057 To

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8050/2/be/src/service/child-query.cc

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 3: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1219/ --

[Impala-ASF-CR] IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 8: Vuk, I have concerns regarding the high-level approach. From a bird's eye

[Impala-ASF-CR] IMPALA-5211: Simplifying nullif conditional.

2017-09-13 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. Change subject: IMPALA-5211: Simplifying nullif conditional. .. Patch Set 7: I should also mention that previous commit messages referred to "ifnull" where they should have

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr local allocations

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5844: use a MemPool for expr local allocations .. Patch Set 6: (2 comments)

[Impala-ASF-CR] IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

2017-09-13 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 8: thanks for the suggestion! I got the sense that the rewrite approach

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() ChildQuery::Cancel() prints a binary ID

[Impala-ASF-CR] IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

2017-09-13 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 8: (5 comments)

[Impala-ASF-CR] IMPALA-5211: Simplifying nullif conditional.

2017-09-13 Thread Philip Zeyliger (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7829 to look at the new patch set (#7). Change subject: IMPALA-5211: Simplifying nullif conditional. .. IMPALA-5211:

[Impala-ASF-CR] IMPALA-5905: build-all-flag-combinations addendum

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/8061 Change subject: IMPALA-5905: build-all-flag-combinations addendum .. IMPALA-5905: build-all-flag-combinations addendum Running the

[Impala-ASF-CR] IMPALA-1649 Pass precision to Print TCounterType::DOUBLE VALUE

2017-09-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. Change subject: IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE .. Patch Set 2: (1 comment)

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

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

[Impala-ASF-CR] IMPALA-5926 : Avoid printing expensive stack when closing a session

2017-09-13 Thread Mostafa Mokhtar (Code Review)
Mostafa Mokhtar has uploaded a new change for review. http://gerrit.cloudera.org:8080/8060 Change subject: IMPALA-5926 : Avoid printing expensive stack when closing a session .. IMPALA-5926 : Avoid printing expensive stack

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. Patch Set 1: (3 comments)

[Impala-ASF-CR] IMPALA-5905: build-all-flag-combinations addendum

2017-09-13 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. Change subject: IMPALA-5905: build-all-flag-combinations addendum .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/8061 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5668: Fix cast(X as timestamp) for negative subsecond Decimals

2017-09-13 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has uploaded a new change for review. http://gerrit.cloudera.org:8080/8051 Change subject: IMPALA-5668: Fix cast(X as timestamp) for negative subsecond Decimals .. IMPALA-5668: Fix cast(X as timestamp) for

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 1: (4 comments) Nice to see this getting fixed.

[Impala-ASF-CR] IMPALA-5926 : Avoid printing expensive stack when closing a session

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5926 : Avoid printing expensive stack when closing a session .. Patch Set 1: Code-Review+1 Amazing. -- To view, visit http://gerrit.cloudera.org:8080/8060

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/8050/3//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-3437: DECIMAL V2: avoid implicit decimal->double conversion

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3437: DECIMAL_V2: avoid implicit decimal->double conversion .. Patch Set 6: (3 comments)

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() ChildQuery::Cancel()

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8050 To

[Impala-ASF-CR] IMPALA-5211: Simplifying nullif conditional.

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5211: Simplifying nullif conditional. .. Patch Set 7: (9 comments)

[Impala-ASF-CR] IMPALA-5926 : Avoid printing expensive stack when closing a session

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5926 : Avoid printing expensive stack when closing a session .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8060 To

[Impala-ASF-CR] IMPALA-5926: Avoid printing expensive stack when closing a session

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5926: Avoid printing expensive stack when closing a session .. Patch Set 2: Code-Review+2 Remove space from commit msg. Carry +2 -- To view, visit

[Impala-ASF-CR] IMPALA-5926: Avoid printing expensive stack when closing a session

2017-09-13 Thread Alex Behm (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8060 to look at the new patch set (#2). Change subject: IMPALA-5926: Avoid printing expensive stack when closing a session

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. Patch Set 1: (3 comments)

[Impala-ASF-CR] IMPALA-5856: Fix outer join predicate assignment.

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5856: Fix outer join predicate assignment. .. Patch Set 1: (5 comments)

[Impala-ASF-CR] IMPALA-5856: Fix outer join predicate assignment.

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#2). Change subject: IMPALA-5856: Fix outer join predicate assignment. .. IMPALA-5856: Fix outer join predicate assignment. Fixes incorrect assignment of join predicates with the

[Impala-ASF-CR] IMPALA-5905: build-all-flag-combinations addendum

2017-09-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5905: build-all-flag-combinations addendum .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8061 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-1649 Pass precision to Print TCounterType::DOUBLE VALUE

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-5416: Fix an impala-shell command recursion bug

2017-09-13 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new change for review. http://gerrit.cloudera.org:8080/8063 Change subject: IMPALA-5416: Fix an impala-shell command recursion bug .. IMPALA-5416: Fix an impala-shell command recursion bug

[Impala-ASF-CR] IMPALA-5926 : Avoid printing expensive stack when closing a session

2017-09-13 Thread Mostafa Mokhtar (Code Review)
Mostafa Mokhtar has abandoned this change. Change subject: IMPALA-5926 : Avoid printing expensive stack when closing a session .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/8060 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/8050/3//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8050/1/be/src/service/child-query.cc

[Impala-ASF-CR] IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel()

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5923: Print binary ID as hex in ChildQuery::Cancel() .. Patch Set 2: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-5844: use a MemPool for expr local allocations

2017-09-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#6). Change subject: IMPALA-5844: use a MemPool for expr local allocations .. IMPALA-5844: use a MemPool for expr local allocations Local allocations in expressions have the following

[Impala-ASF-CR] IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

2017-09-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 8: Our other approach for these small rewrites can be seen in

[Impala-ASF-CR] IMPALA-5890: Abort queries if scanner hits IO errors

2017-09-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5890: Abort queries if scanner hits IO errors .. IMPALA-5890: Abort queries if scanner hits IO errors Prior to this fix, an error in

[Impala-ASF-CR] IMPALA-5890: Abort queries if scanner hits IO errors

2017-09-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5890: Abort queries if scanner hits IO errors .. Patch Set 12: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8011 To unsubscribe,

[Impala-ASF-CR] IMPALA-5597: Try casting targetExpr when building runtime filter plan

2017-09-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5597: Try casting targetExpr when building runtime filter plan .. IMPALA-5597: Try casting targetExpr when building runtime filter plan

[Impala-ASF-CR] IMPALA-5597: Try casting targetExpr when building runtime filter plan

2017-09-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5597: Try casting targetExpr when building runtime filter plan .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-3516: Avoid writing to /tmp in testing

2017-09-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-3516: Avoid writing to /tmp in testing .. IMPALA-3516: Avoid writing to /tmp in testing Currently some parts of the tests write to /tmp:

[Impala-ASF-CR] IMPALA-3516: Avoid writing to /tmp in testing

2017-09-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3516: Avoid writing to /tmp in testing .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8047 To unsubscribe, visit