[Impala-ASF-CR] IMPALA-3641: Fix catalogd RPC responses to DROP TABLE/DATABASE.

2016-12-20 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new change for review. http://gerrit.cloudera.org:8080/5556 Change subject: IMPALA-3641: Fix catalogd RPC responses to DROP TABLE/DATABASE. .. IMPALA-3641: Fix catalogd RPC responses to DROP TABLE/DATA

[Impala-ASF-CR] IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions

2016-12-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5554/2/testdata/bin/check-hbase-

[Impala-ASF-CR] IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions

2016-12-20 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5554/2/testdata/bin/check-hbase-nodes.py Fi

[Impala-ASF-CR] IMPALA-4702: Fix command line help for webserver private key file

2016-12-20 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4702: Fix command line help for webserver_private_key_file .. IMPALA-4702: Fix command line help for webserver_private_key_file The command l

[Impala-ASF-CR] IMPALA-4702: Fix command line help for webserver private key file

2016-12-20 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4702: Fix command line help for webserver_private_key_file .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5552 To unsub

[Impala-ASF-CR] IMPALA-4033: Treat string-partition key values as case sensitive.

2016-12-20 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4033: Treat string-partition key values as case sensitive. .. Patch Set 4: > > Thanks, Amos! I'll take care of running pre-commit tests and > > getting this

[Impala-ASF-CR] IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions

2016-12-20 Thread David Knupp (Code Review)
David Knupp has uploaded a new patch set (#2). Change subject: IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions .. IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions This is the second patch to address IMPALA-4684. T

[Impala-ASF-CR] IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions

2016-12-20 Thread David Knupp (Code Review)
David Knupp has uploaded a new change for review. http://gerrit.cloudera.org:8080/5554 Change subject: IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions .. IMPALA-4684: Handle Zookeeper ConnentionLoss exceptions This is

[Impala-ASF-CR](asf-site) Change git links from GitHub to the official Apache repo.

2016-12-20 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: Change git links from GitHub to the official Apache repo. .. Change git links from GitHub to the official Apache repo. In the discussion on general@incubato

[Impala-ASF-CR](asf-site) Change git links from GitHub to the official Apache repo.

2016-12-20 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Change git links from GitHub to the official Apache repo. .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5433 To unsubscribe, visit ht

[Impala-ASF-CR](asf-site) Change git links from GitHub to the official Apache repo.

2016-12-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Change git links from GitHub to the official Apache repo. .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5433 To unsubscribe, v

[Impala-ASF-CR] IMPALA-4684: Wrap checking for HBase nodes ina try/finally block

2016-12-20 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4684: Wrap checking for HBase nodes ina try/finally block .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5547 To unsub

[Impala-ASF-CR] IMPALA-4684: Wrap checking for HBase nodes ina try/finally block

2016-12-20 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4684: Wrap checking for HBase nodes ina try/finally block .. IMPALA-4684: Wrap checking for HBase nodes ina try/finally block If an excepti

[Impala-ASF-CR] IMPALA-3202,IMPALA-2079: rework scratch file I/O

2016-12-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3202,IMPALA-2079: rework scratch file I/O .. Patch Set 20: do we have end-to-end tests for the blacklist logic? -- To view, visit http://gerrit.cloudera.org

[Impala-ASF-CR] IMPALA-3202,IMPALA-2079: rework scratch file I/O

2016-12-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3202,IMPALA-2079: rework scratch file I/O .. Patch Set 20: (24 comments) http://gerrit.cloudera.org:8080/#/c/5141/20/be/src/runtime/buffered-block-mgr.cc Fil

[Impala-ASF-CR] IMPALA-4033: Treat string-partition key values as case sensitive.

2016-12-20 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4033: Treat string-partition key values as case sensitive. .. Patch Set 4: > Thanks, Amos! I'll take care of running pre-commit tests and > getting this pat

[Impala-ASF-CR](asf-site) Change git links from GitHub to the official Apache repo.

2016-12-20 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#2). Change subject: Change git links from GitHub to the official Apache repo. .. Change git links from GitHub to the official Apache repo. In the discussion on general@incubator around gr

[Impala-ASF-CR] IMPALA-4702: Fix command line help for webserver private key file

2016-12-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4702: Fix command line help for webserver_private_key_file .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5552 To unsub

[Impala-ASF-CR] IMPALA-4702: Fix command line help for webserver private key file

2016-12-20 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4702: Fix command line help for webserver_private_key_file .. Patch Set 1: (1 comment) Thanks for the review. Can you carry the +2? http://gerrit.clouder

[Impala-ASF-CR] IMPALA-4702: Fix command line help for webserver private key file

2016-12-20 Thread Lars Volker (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5552 to look at the new patch set (#2). Change subject: IMPALA-4702: Fix command line help for webserver_private_key_file .

[Impala-ASF-CR] IMPALA-4707: Fix command line help for webserver private key file

2016-12-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4707: Fix command line help for webserver_private_key_file .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5552/1//COMMIT_MSG Commit Mes

[Impala-ASF-CR] IMPALA-4707: Fix command line help for webserver private key file

2016-12-20 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4707: Fix command line help for webserver_private_key_file .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5552 To unsubscri

[Impala-ASF-CR] IMPALA-4707: Fix command line help for webserver private key file

2016-12-20 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/5552 Change subject: IMPALA-4707: Fix command line help for webserver_private_key_file .. IMPALA-4707: Fix command line help for webserver_priv

[Impala-ASF-CR] IMPALA-4033: Treat string-partition key values as case sensitive.

2016-12-20 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4033: Treat string-partition key values as case sensitive. .. Patch Set 4: Code-Review+2 Thanks, Amos! I'll take care of running pre-commit tests and getting

[Impala-ASF-CR] IMPALA-4684: Wrap checking for HBase nodes ina try/finally block

2016-12-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4684: Wrap checking for HBase nodes ina try/finally block .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5547 To unsub

[Impala-ASF-CR] IMPALA-3202,IMPALA-2079: rework scratch file I/O

2016-12-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3202,IMPALA-2079: rework scratch file I/O .. Patch Set 16: (1 comment) http://gerrit.cloudera.org:8080/#/c/5141/16/be/src/runtime/buffered-block-mgr.cc F

[Impala-ASF-CR] IMPALA-3202,IMPALA-2079: rework scratch file I/O

2016-12-20 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3202,IMPALA-2079: rework scratch file I/O .. Patch Set 20: (1 comment) http://gerrit.cloudera.org:8080/#/c/5141/16/be/src/runtime/buffered-block-mgr.cc File

[Impala-ASF-CR] IMPALA-4684: Wrap checking for HBase nodes ina try/finally block

2016-12-20 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-4684: Wrap checking for HBase nodes ina try/finally block .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5547/1/testdata/bin/check-hbase

[Impala-ASF-CR] IMPALA-4684: Wrap checking for HBase nodes ina try/finally block

2016-12-20 Thread David Knupp (Code Review)
David Knupp has uploaded a new patch set (#2). Change subject: IMPALA-4684: Wrap checking for HBase nodes ina try/finally block .. IMPALA-4684: Wrap checking for HBase nodes ina try/finally block If an exception (other than N

[Impala-ASF-CR] IMPALA-4653: fix sticky config variable problem

2016-12-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4653: fix sticky config variable problem .. Patch Set 2: Code-Review+1 Carry +1 -- To view, visit http://gerrit.cloudera.org:8080/5545 To unsubscribe, v

[Impala-ASF-CR] IMPALA-4653: fix sticky config variable problem

2016-12-20 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5545 to look at the new patch set (#2). Change subject: IMPALA-4653: fix sticky config variable problem .. IMPALA-4653: fix

[Impala-ASF-CR] IMPALA-4684: Handle unexpected exceptions while checking for HBase nodes

2016-12-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4684: Handle unexpected exceptions while checking for HBase nodes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5547/1/testdata/bin/ch

[Impala-ASF-CR] IMPALA-3202,IMPALA-2079: rework scratch file I/O

2016-12-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#20). Change subject: IMPALA-3202,IMPALA-2079: rework scratch file I/O .. IMPALA-3202,IMPALA-2079: rework scratch file I/O Refactor BufferedBlockMgr/TmpFileMgr to push more I/O logic i

[Impala-ASF-CR] IMPALA-3202,IMPALA-2079: rework scratch file I/O

2016-12-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3202,IMPALA-2079: rework scratch file I/O .. Patch Set 17: (22 comments) http://gerrit.cloudera.org:8080/#/c/5141/16/be/src/runtime/buffered-block-mgr.cc

[Impala-ASF-CR] IMPALA-4033: Treat string-partition key values as case sensitive.

2016-12-20 Thread Amos Bird (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5535 to look at the new patch set (#4). Change subject: IMPALA-4033: Treat string-partition key values as case sensitive. .

[Impala-ASF-CR] IMPALA-4033: Treat string-partition key values as case sensitive.

2016-12-20 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4033: Treat string-partition key values as case sensitive. .. Patch Set 3: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/5535/3/testdata/wor