[Impala-ASF-CR] IMPALA-4042: Preserve root types when substituting grouping exprs

2016-10-04 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4042: Preserve root types when substituting grouping exprs .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4600 To

[Impala-ASF-CR] IMPALA-4213: Planner not pushing some Kudu predicates

2016-10-04 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#2). Change subject: IMPALA-4213: Planner not pushing some Kudu predicates .. IMPALA-4213: Planner not pushing some Kudu predicates Folding const exprs where there were implicit

[Impala-ASF-CR] IMPALA-3229: Don't assume that AUX exists just because of shell env

2016-10-04 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3229: Don't assume that AUX exists just because of shell env .. Patch Set 4: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-2905: Handle coordinator fragment lifecycle like all others

2016-10-04 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-2905: Handle coordinator fragment lifecycle like all others .. Patch Set 7: (28 comments) This patch passes EE tests. I haven't completely addressed

[Impala-ASF-CR] IMPALA-3983/IMPALA-3974: Delete function jar resources after load

2016-10-04 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3983/IMPALA-3974: Delete function jar resources after load .. Patch Set 2: (1 comment) Any easy way to test this? Can you add some UDFs and then check

[Impala-ASF-CR] IMPALA-4123: Fast bit unpacking

2016-10-04 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4123: Fast bit unpacking .. Patch Set 3: (34 comments) http://gerrit.cloudera.org:8080/#/c/4494/2/be/src/benchmarks/bit-packing-benchmark.cc File

[Impala-ASF-CR] IMPALA-3983/IMPALA-3974: Delete function jar resources after load

2016-10-04 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-3983/IMPALA-3974: Delete function jar resources after load .. Patch Set 2: (1 comment) Thanks Alex. Carrying +1.

[Impala-ASF-CR] IMPALA-3983/IMPALA-3974: Delete function jar resources after load

2016-10-04 Thread Bharath Vissapragada (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4617 to look at the new patch set (#3). Change subject: IMPALA-3983/IMPALA-3974: Delete function jar resources after load

[Impala-ASF-CR] IMPALA-3983: Delete function jar resources after load

2016-10-04 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3983: Delete function jar resources after load .. Patch Set 2: Code-Review+1 (1 comment) Henry, can you check the catalog changes?

[Impala-ASF-CR] IMPALA-3983: Delete function jar resources after load

2016-10-04 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#2). Change subject: IMPALA-3983: Delete function jar resources after load .. IMPALA-3983: Delete function jar resources after load The Catalog copies the UDF jar files to the

[Impala-ASF-CR] IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

2016-10-04 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s .. Patch Set 2: Code-Review+2 Carry +2. -- To view, visit

[Impala-ASF-CR] IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

2016-10-04 Thread Sailesh Mukil (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4622 to look at the new patch set (#2). Change subject: IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

[Impala-ASF-CR] IMPALA-3983: Delete function jar resources after load

2016-10-04 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-3983: Delete function jar resources after load .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4617/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

2016-10-04 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s .. Patch Set 1: Code-Review+2 (2 comments) Carry +2.

[Impala-ASF-CR] IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

2016-10-04 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s .. Patch Set 1: Code-Review+2 (2 comments)

[Impala-ASF-CR] IMPALA-4042: Preserve root types when substituting grouping exprs

2016-10-04 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4042: Preserve root types when substituting grouping exprs .. Patch Set 4: Code-Review+2 Nice work! -- To view, visit http://gerrit.cloudera.org:8080/4600

[Impala-ASF-CR] IMPALA-3983: Delete function jar resources after load

2016-10-04 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3983: Delete function jar resources after load .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4617/1//COMMIT_MSG Commit Message: Line 9:

[Impala-ASF-CR] IMPALA-4123: Fast bit unpacking

2016-10-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). Change subject: IMPALA-4123: Fast bit unpacking .. IMPALA-4123: Fast bit unpacking Adds utility functions for fast unpacking of batches of bit-packed values. These support

[Impala-ASF-CR](asf-site) IMPALA-3808: Add downloads page

2016-10-04 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3808: Add downloads page .. Patch Set 6: Code-Review+2 Just changed commit message, carrying +2 -- To view, visit http://gerrit.cloudera.org:8080/4544 To

[Impala-ASF-CR] IMPALA-1702: Enforce table level consistency accross service

2016-10-04 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1702: Enforce table level consistency accross service .. Patch Set 5: (4 comments)

[Impala-ASF-CR] IMPALA-1670,IMPALA-4141: Support multiple partitions in ALTER TABLE ADD PARTITION

2016-10-04 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-1670,IMPALA-4141: Support multiple partitions in ALTER TABLE ADD PARTITION .. Patch Set 14: (11 comments)

[Impala-ASF-CR] IMPALA-4135: Thrift threaded server times-out connections during high load

2016-10-04 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4135: Thrift threaded server times-out connections during high load .. Patch Set 6: Code-Review+2 (1 comment)

[Impala-ASF-CR] Add vim-specific files to .gitignore

2016-10-04 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: Add vim-specific files to .gitignore .. Add vim-specific files to .gitignore Change-Id: I1abcd8ca0e18178684c916ef6f7d55c25c0814a4 Reviewed-on:

[Impala-ASF-CR] Add vim-specific files to .gitignore

2016-10-04 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: Add vim-specific files to .gitignore .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4562 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4042: count(distinct NULL) fails on a view

2016-10-04 Thread Yonghyun Hwang (Code Review)
Yonghyun Hwang has posted comments on this change. Change subject: IMPALA-4042: count(distinct NULL) fails on a view .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-1702: Enforce table level consistency accross service

2016-10-04 Thread Huaisi Xu (Code Review)
Huaisi Xu has uploaded a new patch set (#5). Change subject: IMPALA-1702: Enforce table level consistency accross service .. IMPALA-1702: Enforce table level consistency accross service Problems: 1.

[Impala-ASF-CR] IMPALA-784: Use `-s in SHOW CREATE TABLE output

2016-10-04 Thread Zoltan Ivanfi (Code Review)
Zoltan Ivanfi has posted comments on this change. Change subject: IMPALA-784: Use `-s in SHOW CREATE TABLE output .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-1702: Enforce table level consistency accross service

2016-10-04 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: IMPALA-1702: Enforce table level consistency accross service .. Patch Set 4: (11 comments)

[Impala-ASF-CR] IMPALA-4135: Thrift threaded server times-out connections during high load

2016-10-04 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4135: Thrift threaded server times-out connections during high load .. Patch Set 6: (3 comments)

[Impala-ASF-CR] IMPALA-889: Add support for an ISO-SQL compliant trim() function.

2016-10-04 Thread Youwei Wang (Code Review)
Youwei Wang has uploaded a new patch set (#8). Change subject: IMPALA-889: Add support for an ISO-SQL compliant trim() function. .. IMPALA-889: Add support for an ISO-SQL compliant trim() function. Purpose: Removes all

[Impala-ASF-CR] IMPALA-889: Add support for an ISO-SQL compliant trim() function.

2016-10-04 Thread Youwei Wang (Code Review)
Youwei Wang has posted comments on this change. Change subject: IMPALA-889: Add support for an ISO-SQL compliant trim() function. .. Patch Set 6: (32 comments) http://gerrit.cloudera.org:8080/#/c/4474/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-889: Add support for an ISO-SQL compliant trim() function.

2016-10-04 Thread Youwei Wang (Code Review)
Youwei Wang has uploaded a new patch set (#7). Change subject: IMPALA-889: Add support for an ISO-SQL compliant trim() function. .. IMPALA-889: Add support for an ISO-SQL compliant trim() function. Purpose: Removes all

[Impala-ASF-CR] IMPALA-3504: UDF for current timestamp in UTC

2016-10-04 Thread Youwei Wang (Code Review)
Youwei Wang has posted comments on this change. Change subject: IMPALA-3504: UDF for current timestamp in UTC .. Patch Set 8: > Carrying the previous +1 so it can be reviewed for a +2. Greetings, dear Matthew. Thank you so