[Impala-ASF-CR] IMPALA-4118: extract encryption utils from BufferedBlockMgr

2016-09-30 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4118: extract encryption utils from BufferedBlockMgr .. Patch Set 12: Code-Review+2 carry +2 -- To view, visit http://gerrit.cloudera.org:8080/4389 To

[Impala-ASF-CR] IMPALA-4118: extract encryption utils from BufferedBlockMgr

2016-09-30 Thread Tim Armstrong (Code Review)
Hello Alex Behm, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4389 to look at the new patch set (#11). Change subject: IMPALA-4118: extract encryption utils from BufferedBlockMgr

[Impala-ASF-CR] IMPALA-4118: extract encryption utils from BufferedBlockMgr

2016-09-30 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4118: extract encryption utils from BufferedBlockMgr .. Patch Set 12: (3 comments)

[Impala-ASF-CR] IMPALA-4026: Implement double-buffering for BlockingQueue

2016-09-30 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4026: Implement double-buffering for BlockingQueue .. IMPALA-4026: Implement double-buffering for BlockingQueue With recent changes to

[Impala-ASF-CR] IMPALA-4026: Implement double-buffering for BlockingQueue

2016-09-30 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4026: Implement double-buffering for BlockingQueue .. Patch Set 12: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4350 To unsubscribe,

[Impala-ASF-CR] IMPALA-4216: Test became flaky: TestTpchMemLimitError.test low mem limit q20

2016-09-30 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new change for review. http://gerrit.cloudera.org:8080/4572 Change subject: IMPALA-4216: Test became flaky: TestTpchMemLimitError.test_low_mem_limit_q20 .. IMPALA-4216: Test became flaky:

[Impala-ASF-CR] IMPALA-4118: extract encryption utils from BufferedBlockMgr

2016-09-30 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4118: extract encryption utils from BufferedBlockMgr .. IMPALA-4118: extract encryption utils from BufferedBlockMgr As groundwork for

[Impala-ASF-CR] IMPALA-4118: extract encryption utils from BufferedBlockMgr

2016-09-30 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4118: extract encryption utils from BufferedBlockMgr .. Patch Set 12: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4389 To

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

2016-09-30 Thread Youwei Wang (Code Review)
Youwei Wang has uploaded a new patch set (#6). 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] Don't assume that AUX exists just because a shell variable is set

2016-09-30 Thread Jim Apple (Code Review)
Hello Michael Brown, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4563 to look at the new patch set (#2). Change subject: Don't assume that AUX exists just because a shell variable is set

[Impala-ASF-CR] IMPALA-4196: Cross compile bit-byte-functions

2016-09-30 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4196: Cross compile bit-byte-functions .. Patch Set 3: (1 comment)

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

2016-09-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#5). Change subject: IMPALA-4135: Thrift threaded server times-out connections during high load .. IMPALA-4135: Thrift threaded server times-out connections during high load

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-30 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Don't assume that AUX exists just because a shell variable is set .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-4234: Remove astyle config file, looks outdated.

2016-09-30 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/4578 Change subject: IMPALA-4234: Remove astyle config file, looks outdated. .. IMPALA-4234: Remove astyle config file, looks outdated.

[Impala-ASF-CR] IMPALA-4196: Cross compile bit-byte-functions

2016-09-30 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4196: Cross compile bit-byte-functions .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-4196: Cross compile bit-byte-functions

2016-09-30 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4196: Cross compile bit-byte-functions .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-4196: Cross compile bit-byte-functions

2016-09-30 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4196: Cross compile bit-byte-functions .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-4196: Cross compile bit-byte-functions

2016-09-30 Thread Bharath Vissapragada (Code Review)
Hello Michael Ho, Dan Hecht, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4557 to look at the new patch set (#4). Change subject: IMPALA-4196: Cross compile bit-byte-functions

[Impala-ASF-CR] IMPALA-4196: Cross compile bit-byte-functions

2016-09-30 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4196: Cross compile bit-byte-functions .. Patch Set 4: Code-Review+2 Rebased. -- To view, visit http://gerrit.cloudera.org:8080/4557 To

[Impala-ASF-CR] IMPALA-4230: ASF policy issues from 2.7.0 rc3.

2016-09-30 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4230: ASF policy issues from 2.7.0 rc3. .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4582/1//COMMIT_MSG Commit Message: PS1,

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

2016-09-30 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 3: (10 comments)

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-30 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: Don't assume that AUX exists just because a shell variable is set .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-4232: qgen: Hive does not support aggregates inside specific analytic clauses

2016-09-30 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4232: qgen: Hive does not support aggregates inside specific analytic clauses .. Patch Set 2: (5 comments)

[Impala-ASF-CR] IMPALA-4196: Cross compile bit-byte-functions

2016-09-30 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4196: Cross compile bit-byte-functions .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4557 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4196: Cross compile bit-byte-functions

2016-09-30 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4196: Cross compile bit-byte-functions .. IMPALA-4196: Cross compile bit-byte-functions Change-Id: I5a1291bfd202b500405a884e4a62f0ca2447244a

[Impala-ASF-CR] IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables

2016-09-30 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new patch set (#3). Change subject: IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables .. IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables With this commit we

[Impala-ASF-CR] IMPALA-4230: ASF policy issues from 2.7.0 rc3.

2016-09-30 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#2). Change subject: IMPALA-4230: ASF policy issues from 2.7.0 rc3. .. IMPALA-4230: ASF policy issues from 2.7.0 rc3. In our IPMC vote to release 2.7.0 rc3, Justing Mclean pointed out a

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

2016-09-30 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: (7 comments) http://gerrit.cloudera.org:8080/#/c/4490/6/be/src/exprs/expr-test.cc File

[Impala-ASF-CR] IMPALA-4232: qgen: Hive does not support aggregates inside specific analytic clauses

2016-09-30 Thread Sahil Takiar (Code Review)
Sahil Takiar has uploaded a new change for review. http://gerrit.cloudera.org:8080/4581 Change subject: IMPALA-4232: qgen: Hive does not support aggregates inside specific analytic clauses .. IMPALA-4232: qgen: Hive does not

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

2016-09-30 Thread Alex Behm (Code Review)
Alex Behm 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-3725 Support Kudu UPSERT in Impala

2016-09-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3725 Support Kudu UPSERT in Impala .. Patch Set 5: The latest version moves back to using a single class, InsertStmt, rather than having

[Impala-ASF-CR] IMPALA-4230: ASF policy issues from 2.7.0 rc3.

2016-09-30 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4230: ASF policy issues from 2.7.0 rc3. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4582/2/llvm-ir/test-loop.bc File

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

2016-09-30 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: (4 comments) http://gerrit.cloudera.org:8080/#/c/4474/3//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-4237: Fix materialization of 4 byte decimals in data source scan node.

2016-09-30 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new change for review. http://gerrit.cloudera.org:8080/4585 Change subject: IMPALA-4237: Fix materialization of 4 byte decimals in data source scan node. .. IMPALA-4237: Fix materialization of 4 byte

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

2016-09-30 Thread Lars Volker (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4562 to look at the new patch set (#2). Change subject: Add vim-specific files to .gitignore .. Add vim-specific files

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

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

[Impala-ASF-CR] IMPALA-4118: extract encryption utils from BufferedBlockMgr

2016-09-30 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4118: extract encryption utils from BufferedBlockMgr .. Patch Set 13: (1 comment)