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

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8122 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 6: Verified+1 -- To

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

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8122 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. IMPALA-1767 Adds predicate

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

2017-09-22 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8132 ) Change subject: IMPALA-5975: Work around broken beeline clients .. Patch Set 1: Code-Review+2 Omg, sorry you had to dig into this -- To

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

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8029 ) Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 6: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8084 ) Change subject: IMPALA-5599: Fix for mis-use of TimestampValue .. IMPALA-5599: Fix for mis-use of TimestampValue The

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8084 ) Change subject: IMPALA-5599: Fix for mis-use of TimestampValue .. Patch Set 7: Verified+1 -- To view, visit

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

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8004 ) Change subject: IMPALA-4513: Promote integer types for ABS() .. Patch Set 7: Verified+1 -- To view, visit

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

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8004 ) Change subject: IMPALA-4513: Promote integer types for ABS() .. IMPALA-4513: Promote integer types for ABS() The

[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 Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8132 ) Change subject: IMPALA-5975: Work around broken beeline clients .. Patch Set 1: Code-Review+1 This seems fine to me. Have we filed a

[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-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Hello Matthew Mulder, Michael Brown, David Knupp, Alex Behm, Mostafa Mokhtar, Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8102 to look at the new patch set (#8). Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for

[Impala-ASF-CR] Remove Lamar County record from expected, not seen on public Jenkins (unless flapping).

2017-09-22 Thread Tim Wood (Code Review)
Tim Wood has abandoned this change. ( http://gerrit.cloudera.org:8080/8131 ) Change subject: Remove Lamar County record from expected, not seen on public Jenkins (unless flapping). .. Abandoned Forgot to rebase into original

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Hello Matthew Mulder, Michael Brown, David Knupp, Alex Behm, Mostafa Mokhtar, Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8102 to look at the new patch set (#7). Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for

[Impala-ASF-CR] Remove Lamar County record from expected, not seen on public Jenkins (unless flapping).

2017-09-22 Thread Tim Wood (Code Review)
Tim Wood has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8131 Change subject: Remove Lamar County record from expected, not seen on public Jenkins (unless flapping). .. Remove Lamar County record from

[Impala-ASF-CR] IMPALA-5870: Improve runtime profile for partial sort

2017-09-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8123 ) Change subject: IMPALA-5870: Improve runtime profile for partial sort .. Patch Set 2: Code-Review+2 -- To view, visit

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

2017-09-22 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8122 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 6: Code-Review+2 -- To view, visit

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

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8122 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 6: Build started:

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/8122 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 6: (1 comment)

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8122 to look at the new patch set (#6). Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

[Impala-ASF-CR] IMPALA-5870: Improve runtime profile for partial sort

2017-09-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8123 ) Change subject: IMPALA-5870: Improve runtime profile for partial sort .. Patch Set 2: (2 comments)

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

2017-09-22 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8122 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 5: Code-Review+2 (1 comment) I'll

[Impala-ASF-CR] IMPALA-5870: Improve runtime profile for partial sort

2017-09-22 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8123 to look at the new patch set (#2). Change subject: IMPALA-5870: Improve runtime profile for partial sort ..

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

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8029 ) Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 6: Build started:

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/8122 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 5: (1 comment)

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8122 to look at the new patch set (#5). Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

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

2017-09-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr local allocations .. Patch Set 6: (3 comments) I guess my vote is for

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

2017-09-22 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8122 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 4: (1 comment) Nice, almost done

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8084 ) Change subject: IMPALA-5599: Fix for mis-use of TimestampValue .. Patch Set 7: Build started:

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8084 ) Change subject: IMPALA-5599: Fix for mis-use of TimestampValue .. Patch Set 7: Code-Review+2 -- To view, visit

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has abandoned this change. ( http://gerrit.cloudera.org:8080/8105 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Abandoned Abandoning in favor of

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has abandoned this change. ( http://gerrit.cloudera.org:8080/8014 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Abandoned Abandoning in favor of

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

2017-09-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8004 ) Change subject: IMPALA-4513: Promote integer types for ABS() .. Patch Set 7: Build started:

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

2017-09-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8029 ) Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 6: Code-Review+2 -- To view, visit

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

2017-09-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8004 ) Change subject: IMPALA-4513: Promote integer types for ABS() .. Patch Set 7: Code-Review+2 -- To view, visit

[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-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8102 ) Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for Impala. .. Patch Set 6: (1 comment)

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8102 ) Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for Impala. .. Patch Set 6: (1 comment) > Patch Set 5: > > (1

[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-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. ( http://gerrit.cloudera.org:8080/8102 ) Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for Impala. .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8102 ) Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for Impala. .. Patch Set 6: (1 comment)

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

2017-09-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8029 ) Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 2: (4 comments)

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

2017-09-22 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Matthew Jacobs, Jim Apple, Philip Zeyliger, Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8029 to look at the new patch set (#6). Change subject: IMPALA-3360: Codegen inserting into runtime filters

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Zoram Thanga (Code Review)
Zoram Thanga has posted comments on this change. ( http://gerrit.cloudera.org:8080/8084 ) Change subject: IMPALA-5599: Fix for mis-use of TimestampValue .. Patch Set 6: (1 comment) Changed to C++ static_cast.

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Zoram Thanga (Code Review)
Hello Matthew Jacobs, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8084 to look at the new patch set (#7). Change subject: IMPALA-5599: Fix for mis-use of TimestampValue ..

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8102 ) Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for Impala. .. Patch Set 6: (2 comments)

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8102 ) Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for Impala. .. Patch Set 6: (4 comments) Updates to PS 5.

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Hello Matthew Mulder, Michael Brown, David Knupp, Alex Behm, Mostafa Mokhtar, Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8102 to look at the new patch set (#6). Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8084 ) Change subject: IMPALA-5599: Fix for mis-use of TimestampValue .. Patch Set 6: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8102 ) Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for Impala. .. Patch Set 5: (7 comments)

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

2017-09-22 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8056 to look at the new patch set (#4). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Zoram Thanga (Code Review)
Hello Matthew Jacobs, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8084 to look at the new patch set (#6). Change subject: IMPALA-5599: Fix for mis-use of TimestampValue ..

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Zoram Thanga (Code Review)
Zoram Thanga has posted comments on this change. ( http://gerrit.cloudera.org:8080/8084 ) Change subject: IMPALA-5599: Fix for mis-use of TimestampValue .. Patch Set 5: (1 comment) Fixed the test code as suggested. Please

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8102 ) Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for Impala. .. Patch Set 5: PS 5 removes old comments from

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Hello Matthew Mulder, Michael Brown, David Knupp, Alex Behm, Mostafa Mokhtar, Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8102 to look at the new patch set (#5). Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for

[Impala-ASF-CR] IMPALA-5307: part 1: don't transfer disk I/O buffers out of parquet

2017-09-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/8085 ) Change subject: IMPALA-5307: part 1: don't transfer disk I/O buffers out of parquet .. IMPALA-5307: part 1: don't transfer disk I/O

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8102 ) Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for Impala. .. Patch Set 3: PS 4 removes some wayward tab chars.

[Impala-ASF-CR] IMPALA-5870: Improve explain/profile output for partial sort

2017-09-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8123 ) Change subject: IMPALA-5870: Improve explain/profile output for partial sort .. Patch Set 1: (3 comments) The profile changes look

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Hello Matthew Mulder, Michael Brown, David Knupp, Alex Behm, Mostafa Mokhtar, Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8102 to look at the new patch set (#3). Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for

[Impala-ASF-CR](asf-site) Change log for 2.10.0 release

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8079 ) Change subject: Change log for 2.10.0 release .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8079/2/impala-docs.html

[Impala-ASF-CR] IMPALA-5376: Implement all TPCDS test cases or alternates for Impala.

2017-09-22 Thread Tim Wood (Code Review)
Hello Matthew Mulder, Michael Brown, David Knupp, Alex Behm, Mostafa Mokhtar, Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8102 to look at the new patch set (#2). Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for

[Impala-ASF-CR](asf-site) Change log for 2.10.0 release

2017-09-22 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8079 ) Change subject: Change log for 2.10.0 release .. Patch Set 2: (1 comment)

[Impala-ASF-CR](asf-site) revive link to HTML docs for 2.9

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8126 ) Change subject: revive link to HTML docs for 2.9 .. Patch Set 2: Verified+1 verified by hosting the site locally and clicking through

[Impala-ASF-CR](asf-site) revive link to HTML docs for 2.9

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8126 ) Change subject: revive link to HTML docs for 2.9 .. revive link to HTML docs for 2.9 The change "Change log for 2.10.0

[Impala-ASF-CR](asf-site) revive link to HTML docs for 2.9

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8126 ) Change subject: revive link to HTML docs for 2.9 .. Patch Set 2: Code-Review+2 carry +2 -- To view, visit

[Impala-ASF-CR](asf-site) revive link to HTML docs for 2.9

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8126 ) Change subject: revive link to HTML docs for 2.9 .. Patch Set 1: (1 comment) Thank you!

[Impala-ASF-CR](asf-site) revive link to HTML docs for 2.9

2017-09-22 Thread Michael Brown (Code Review)
Hello Bharath Vissapragada, Greg Rahn, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8126 to look at the new patch set (#2). Change subject: revive link to HTML docs for 2.9 .. revive

[Impala-ASF-CR](asf-site) revive link to HTML docs for 2.9

2017-09-22 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8126 ) Change subject: revive link to HTML docs for 2.9 .. Patch Set 1: (1 comment)

[Impala-ASF-CR](asf-site) revive link to HTML docs for 2.9

2017-09-22 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8126 ) Change subject: revive link to HTML docs for 2.9 .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR](asf-site) Change log for 2.10.0 release

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8079 ) Change subject: Change log for 2.10.0 release .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8079/2/impala-docs.html

[Impala-ASF-CR](asf-site) Change log for 2.10.0 release

2017-09-22 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8079 ) Change subject: Change log for 2.10.0 release .. Patch Set 2: (1 comment)

[Impala-ASF-CR](asf-site) revive link to HTML docs for 2.9

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8126 Change subject: revive link to HTML docs for 2.9 .. revive link to HTML docs for 2.9 The change "Change log for 2.10.0 release" removed

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

2017-09-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr local allocations .. Patch Set 6: (4 comments)

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

2017-09-22 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8025 to look at the new patch set (#7). Change subject: IMPALA-5844: use a MemPool for expr local allocations ..

[Impala-ASF-CR](asf-site) Change log for 2.10.0 release

2017-09-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8079 ) Change subject: Change log for 2.10.0 release .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8079/2/impala-docs.html

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

2017-09-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8025 ) Change subject: IMPALA-5844: use a MemPool for expr local allocations .. Patch Set 6: (3 comments)

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8084 ) Change subject: IMPALA-5599: Fix for mis-use of TimestampValue .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-5538: Use explicit catalog versions for deleted objects

2017-09-22 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/7731 ) Change subject: IMPALA-5538: Use explicit catalog versions for deleted objects .. Patch Set 6: (4 comments)

[Impala-ASF-CR] IMPALA-5664: Unix time to timestamp conversions may crash Impala

2017-09-22 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/7954 ) Change subject: IMPALA-5664: Unix time to timestamp conversions may crash Impala .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Zoram Thanga (Code Review)
Hello Matthew Jacobs, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8084 to look at the new patch set (#5). Change subject: IMPALA-5599: Fix for mis-use of TimestampValue ..

[Impala-ASF-CR] IMPALA-5599: Fix for mis-use of TimestampValue

2017-09-22 Thread Zoram Thanga (Code Review)
Zoram Thanga has posted comments on this change. ( http://gerrit.cloudera.org:8080/8084 ) Change subject: IMPALA-5599: Fix for mis-use of TimestampValue .. Patch Set 4: (1 comment) Made the last suggested change.

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/8122 ) Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 4: (11 comments)

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8122 to look at the new patch set (#4). Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8122 to look at the new patch set (#3). Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

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

2017-09-22 Thread Vuk Ercegovac (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8122 to look at the new patch set (#2). Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown.