[Impala-ASF-CR] IMPALA-8584: Add cookie support to the HTTP HS2 server

2019-06-18 Thread Thomas Marshall (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13672 to look at the new patch set (#2). Change subject: IMPALA-8584: Add cookie support to the HTTP HS2 server

[Impala-ASF-CR] IMPALA-8584: Add cookie support to the HTTP HS2 server

2019-06-18 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13672 Change subject: IMPALA-8584: Add cookie support to the HTTP HS2 server .. IMPALA-8584: Add cookie support to the HTTP HS2 server

[Impala-ASF-CR] IMPALA-7467: Ported ExecQueryFInstances over to krpc

2019-06-17 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13583 ) Change subject: IMPALA-7467: Ported ExecQueryFInstances over to krpc .. Patch Set 2: (8 comments)

[Impala-ASF-CR] IMPALA-7467: Ported ExecQueryFInstances over to krpc

2019-06-17 Thread Thomas Marshall (Code Review)
Hello Michael Ho, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13583 to look at the new patch set (#2). Change subject: IMPALA-7467: Ported ExecQueryFInstances over to krpc

[Impala-ASF-CR] IMPALA-7290: part 2: Add HS2 support to Impala shell

2019-06-14 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/12884 ) Change subject: IMPALA-7290: part 2: Add HS2 support to Impala shell .. Patch Set 22: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-8583: Add metrics for Basic auth

2019-06-13 Thread Thomas Marshall (Code Review)
Hello Todd Lipcon, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13640 to look at the new patch set (#3). Change subject: IMPALA-8583: Add metrics for Basic auth ..

[Impala-ASF-CR] IMPALA-8583: Add metrics for BASIC auth

2019-06-13 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13640 ) Change subject: IMPALA-8583: Add metrics for BASIC auth .. Patch Set 2: (6 comments)

[Impala-ASF-CR] IMPALA-8583: Add metrics for BASIC auth

2019-06-13 Thread Thomas Marshall (Code Review)
Hello Todd Lipcon, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13640 to look at the new patch set (#2). Change subject: IMPALA-8583: Add metrics for BASIC auth ..

[Impala-ASF-CR] IMPALA-8583: Add metrics for BASIC auth

2019-06-13 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13640 Change subject: IMPALA-8583: Add metrics for BASIC auth .. IMPALA-8583: Add metrics for BASIC auth The patch adds two metrics:

[Impala-ASF-CR] IMPALA-8629: (part 2) Adjust new KuduStorageHandler package

2019-06-12 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13541 ) Change subject: IMPALA-8629: (part 2) Adjust new KuduStorageHandler package .. Patch Set 9: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-8629: (part 2) Adjust new KuduStorageHandler package

2019-06-12 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13541 ) Change subject: IMPALA-8629: (part 2) Adjust new KuduStorageHandler package .. Patch Set 8: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7467: Ported ExecQueryFInstances over to krpc

2019-06-11 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13583 Change subject: IMPALA-7467: Ported ExecQueryFInstances over to krpc .. IMPALA-7467: Ported ExecQueryFInstances over to krpc This

[Impala-ASF-CR] IMPALA-8623: Expose HS2 HTTP port in containers

2019-06-10 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13520 ) Change subject: IMPALA-8623: Expose HS2 HTTP port in containers .. Patch Set 2: Code-Review+2 (1 comment)

[Impala-ASF-CR] Add option to set driver in ImpalaJdbcDriver

2019-06-10 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13514 ) Change subject: Add option to set driver in ImpalaJdbcDriver .. Patch Set 4: Code-Review+2 gvo failure was unrelated -- To view,

[Impala-ASF-CR] IMPALA-8623: Expose HS2 HTTP port in containers

2019-06-10 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13520 ) Change subject: IMPALA-8623: Expose HS2 HTTP port in containers .. Patch Set 2: Did you still want to get this in, given the

[Impala-ASF-CR] IMPALA-8629: (part 1) Add temp KuduStorageHandler

2019-06-10 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13561 ) Change subject: IMPALA-8629: (part 1) Add temp KuduStorageHandler .. IMPALA-8629: (part 1) Add temp KuduStorageHandler This

[Impala-ASF-CR] IMPALA-8629: (part 1) Add temp KuduStorageHandler

2019-06-10 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13561 ) Change subject: IMPALA-8629: (part 1) Add temp KuduStorageHandler .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-8635. Use local metastore URIs when checking for Kudu/HMS integration

2019-06-07 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13555 ) Change subject: IMPALA-8635. Use local metastore URIs when checking for Kudu/HMS integration .. IMPALA-8635. Use local

[Impala-ASF-CR] Add option to set driver in ImpalaJdbcDriver

2019-06-07 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13514 ) Change subject: Add option to set driver in ImpalaJdbcDriver .. Patch Set 2: Yeah, it was a draft, should be published now -- To

[Impala-ASF-CR] Add option to set driver in ImpalaJdbcDriver

2019-06-07 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/13514 ) Change subject: Add option to set driver in ImpalaJdbcDriver .. Add option to set driver in ImpalaJdbcDriver ImpalaJdbcDriver is a

[Impala-ASF-CR] IMPALA-8629: (part 1) Add temp KuduStorageHandler

2019-06-07 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13561 ) Change subject: IMPALA-8629: (part 1) Add temp KuduStorageHandler .. Patch Set 2: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-8635. Use local metastore URIs when checking for Kudu/HMS integration

2019-06-07 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13555 ) Change subject: IMPALA-8635. Use local metastore URIs when checking for Kudu/HMS integration .. Patch Set 1: Code-Review+2 -- To

[Impala-ASF-CR] IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration

2019-06-06 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13409 ) Change subject: IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration .. IMPALA-8506: Support RENAME TABLE

[Impala-ASF-CR] IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration

2019-06-06 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13409 ) Change subject: IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration .. Patch Set 5: Code-Review+2 -- To view,

[Impala-ASF-CR] Fix integration of kudu-hive.jar

2019-06-06 Thread Thomas Marshall (Code Review)
Thomas Marshall has abandoned this change. ( http://gerrit.cloudera.org:8080/13544 ) Change subject: Fix integration of kudu-hive.jar .. Abandoned pushed by mistake -- To view, visit http://gerrit.cloudera.org:8080/13544 To

[Impala-ASF-CR] Fix integration of kudu-hive.jar

2019-06-06 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13542 ) Change subject: Fix integration of kudu-hive.jar .. Patch Set 4: Code-Review+2 carrying forward -- To view, visit

[Impala-ASF-CR] Fix integration of kudu-hive.jar

2019-06-06 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13544 Change subject: Fix integration of kudu-hive.jar .. Fix integration of kudu-hive.jar IMPALA-8503 added downloading kudu-hive.jar and

[Impala-ASF-CR] Fix integration of kudu-hive.jar

2019-06-06 Thread Thomas Marshall (Code Review)
Hello Lars Volker, Hao Hao, Csaba Ringhofer, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13542 to look at the new patch set (#4). Change subject: Fix integration of kudu-hive.jar

[Impala-ASF-CR] Fix integration of kudu-hive.jar

2019-06-06 Thread Thomas Marshall (Code Review)
Hello Lars Volker, Hao Hao, Csaba Ringhofer, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13542 to look at the new patch set (#3). Change subject: Fix integration of kudu-hive.jar

[Impala-ASF-CR] Fix integration of kudu-hive.jar

2019-06-06 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13542 ) Change subject: Fix integration of kudu-hive.jar .. Patch Set 3: (2 comments)

[Impala-ASF-CR] Fix integration of kudu-hive.jar

2019-06-06 Thread Thomas Marshall (Code Review)
Hello Hao Hao, Csaba Ringhofer, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13542 to look at the new patch set (#2). Change subject: Fix integration of kudu-hive.jar

[Impala-ASF-CR] Improve adding kudu-hive.jar to HADOOP CLASSPATH

2019-06-06 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13542 Change subject: Improve adding kudu-hive.jar to HADOOP_CLASSPATH .. Improve adding kudu-hive.jar to HADOOP_CLASSPATH IMPALA-8503

[Impala-ASF-CR] IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration

2019-06-05 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13409 ) Change subject: IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration

2019-06-05 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13409 ) Change subject: IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration

2019-06-05 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13409 ) Change subject: IMPALA-8506: Support RENAME TABLE statement with Kudu/HMS integration .. Patch Set 3: (3 comments)

[Impala-ASF-CR] IMPALA-8507: Support DROP TABLE statement with Kudu/HMS integration

2019-06-05 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13400 ) Change subject: IMPALA-8507: Support DROP TABLE statement with Kudu/HMS integration .. IMPALA-8507: Support DROP TABLE

[Impala-ASF-CR] IMPALA-8507: Support DROP TABLE statement with Kudu/HMS integration

2019-06-05 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13400 ) Change subject: IMPALA-8507: Support DROP TABLE statement with Kudu/HMS integration .. Patch Set 8: Code-Review+2 When the verify

[Impala-ASF-CR] Add option to set driver in ImpalaJdbcDriver

2019-06-04 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13514 Change subject: Add option to set driver in ImpalaJdbcDriver .. Add option to set driver in ImpalaJdbcDriver ImpalaJdbcDriver is a

[Impala-ASF-CR] IMPALA-8505: Disallow setting Kudu table id in ALTER TABLE

2019-06-04 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13465 ) Change subject: IMPALA-8505: Disallow setting Kudu table id in ALTER TABLE .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-8507: Support DROP TABLE statement with Kudu/HMS integration

2019-06-04 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13400 ) Change subject: IMPALA-8507: Support DROP TABLE statement with Kudu/HMS integration .. Patch Set 7: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration

2019-06-04 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13375 ) Change subject: IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration .. IMPALA-8504 (part 2):

[Impala-ASF-CR] IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration

2019-06-04 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13375 ) Change subject: IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 9: Verified+1 There

[Impala-ASF-CR] IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration

2019-06-04 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13375 ) Change subject: IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 8: Code-Review+2 --

[Impala-ASF-CR] IMPALA-8507: Support DROP TABLE statement with Kudu/HMS integration

2019-06-03 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13400 ) Change subject: IMPALA-8507: Support DROP TABLE statement with Kudu/HMS integration .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration

2019-06-03 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13375 ) Change subject: IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 6: Code-Review+2 --

[Impala-ASF-CR] IMPALA-8603: remove confusing logging from custom cluster tests

2019-06-03 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13502 Change subject: IMPALA-8603: remove confusing logging from custom cluster tests .. IMPALA-8603: remove confusing logging from custom

[Impala-ASF-CR] IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration

2019-06-03 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13375 ) Change subject: IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-8503: allow the Hive Metastore to start with kudu-hive plugin

2019-06-03 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13319 ) Change subject: IMPALA-8503: allow the Hive Metastore to start with kudu-hive plugin .. IMPALA-8503: allow the Hive

[Impala-ASF-CR] IMPALA-8503: allow the Hive Metastore to start with kudu-hive plugin

2019-06-03 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13319 ) Change subject: IMPALA-8503: allow the Hive Metastore to start with kudu-hive plugin .. Patch Set 5: Code-Review+2 Thanks! -- To

[Impala-ASF-CR] Disable cumstom cluster/service FE tests on S3

2019-06-03 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13500 ) Change subject: Disable cumstom cluster/service FE tests on S3 .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13375 ) Change subject: IMPALA-8504 (part 2): Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 4: (6 comments)

[Impala-ASF-CR] IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13318 ) Change subject: IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration .. IMPALA-8504: Support CREATE TABLE

[Impala-ASF-CR] IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13318 ) Change subject: IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 13: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-8503: allow the Hive Metastore to start with kudu-hive plugin

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13319 ) Change subject: IMPALA-8503: allow the Hive Metastore to start with kudu-hive plugin .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-8502: Bump CDH BUILD NUMBER and Kudu version

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13481 ) Change subject: IMPALA-8502: Bump CDH_BUILD_NUMBER and Kudu version .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-8503: allow the Hive Metastore to start with kudu-hive plugin

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13319 ) Change subject: IMPALA-8503: allow the Hive Metastore to start with kudu-hive plugin .. Patch Set 2: (1 comment)

[native-toolchain-CR] Bump Kudu version to 84086fe

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13479 ) Change subject: Bump Kudu version to 84086fe .. Bump Kudu version to 84086fe This pulls in changes needed for the Impala

[native-toolchain-CR] Bump Kudu version to 84086fe

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13479 ) Change subject: Bump Kudu version to 84086fe .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/13479 To

[native-toolchain-CR] Bump Kudu version to 84086fe

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13479 ) Change subject: Bump Kudu version to 84086fe .. Patch Set 2: see https://gerrit.cloudera.org/#/c/13481/ -- To view, visit

[native-toolchain-CR] Bump Kudu version to 84086fe

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/13479 ) Change subject: Bump Kudu version to 84086fe .. Bump Kudu version to 84086fe This pulls in changes needed for the Impala side of the

[Impala-ASF-CR] IMPALA-8502: Bump CDH BUILD NUMBER and Kudu version

2019-05-31 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13481 ) Change subject: IMPALA-8502: Bump CDH_BUILD_NUMBER and Kudu version .. Patch Set 3: Code-Review+2 -- To view, visit

[native-toolchain-CR] Bump Kudu version to 84086fe

2019-05-30 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13479 Change subject: Bump Kudu version to 84086fe .. Bump Kudu version to 84086fe Change-Id: Ie63a8baf1564f63d6ec9c0b953e2fa4119f51190

[Impala-ASF-CR] IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint

2019-05-29 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13299 ) Change subject: IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint .. Patch Set 6: Code-Review+2 rebased on top

[Impala-ASF-CR] IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint

2019-05-29 Thread Thomas Marshall (Code Review)
Hello Bharath Vissapragada, Michael Ho, Sudhanshu Arora, Mike Yoder, Todd Lipcon, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13299 to look at the new patch set (#6). Change subject: IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based

[Impala-ASF-CR] IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration

2019-05-29 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13318 ) Change subject: IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 8: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-29 Thread Thomas Marshall (Code Review)
Hello Michael Ho, Andrew Sherman, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13306 to look at the new patch set (#8). Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-29 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13306 ) Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed .. Patch Set 8: (4 comments)

[Impala-ASF-CR] IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration

2019-05-29 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13318 ) Change subject: IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 7: (5 comments) Looks good,

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-29 Thread Thomas Marshall (Code Review)
Hello Michael Ho, Andrew Sherman, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13306 to look at the new patch set (#9). Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-29 Thread Thomas Marshall (Code Review)
Hello Michael Ho, Andrew Sherman, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13306 to look at the new patch set (#10). Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-29 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13306 ) Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed .. Patch Set 10: (4 comments)

[Impala-ASF-CR] IMPALA-8538 (part 1) Copied THttp(Server|Transport) from thrift-0.9.3

2019-05-28 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13298 ) Change subject: IMPALA-8538 (part 1) Copied THttp(Server|Transport) from thrift-0.9.3 .. Patch Set 2: Code-Review+2 see:

[Impala-ASF-CR] IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint

2019-05-28 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13299 ) Change subject: IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint .. Patch Set 5: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint

2019-05-28 Thread Thomas Marshall (Code Review)
Hello Bharath Vissapragada, Michael Ho, Sudhanshu Arora, Mike Yoder, Todd Lipcon, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13299 to look at the new patch set (#5). Change subject: IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based

[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-28 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13404 ) Change subject: IMPALA-8333: Remove Impala Shell warnings part 2 .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration

2019-05-24 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13318 ) Change subject: IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 5: (2 comments)

[Impala-ASF-CR] IMPALA-8570: Fix flakiness in test restart statestore query resilience

2019-05-24 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13406 ) Change subject: IMPALA-8570: Fix flakiness in test_restart_statestore_query_resilience .. Patch Set 2: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-8503: add option to start Kudu cluster with HMS integration

2019-05-24 Thread Thomas Marshall (Code Review)
Thomas Marshall has abandoned this change. ( http://gerrit.cloudera.org:8080/13248 ) Change subject: IMPALA-8503: add option to start Kudu cluster with HMS integration .. Abandoned see https://gerrit.cloudera.org/#/c/13318/

[Impala-ASF-CR] IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint

2019-05-23 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13299 ) Change subject: IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint .. Patch Set 4: (12 comments)

[Impala-ASF-CR] IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint

2019-05-23 Thread Thomas Marshall (Code Review)
Hello Bharath Vissapragada, Michael Ho, Sudhanshu Arora, Mike Yoder, Todd Lipcon, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13299 to look at the new patch set (#4). Change subject: IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based

[Impala-ASF-CR] IMPALA-4271: Introduce the new Kudu storage handler

2019-05-23 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13358 ) Change subject: IMPALA-4271: Introduce the new Kudu storage handler .. Patch Set 7: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-23 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13306 ) Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed .. Patch Set 7: (1 comment)

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-23 Thread Thomas Marshall (Code Review)
Hello Michael Ho, Andrew Sherman, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13306 to look at the new patch set (#7). Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed

[Impala-ASF-CR] IMPALA-8504: Introduce the new Kudu storage handler

2019-05-23 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13358 ) Change subject: IMPALA-8504: Introduce the new Kudu storage handler .. Patch Set 6: If you'll address the previous comment about

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-23 Thread Thomas Marshall (Code Review)
Hello Michael Ho, Andrew Sherman, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13306 to look at the new patch set (#6). Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-23 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13306 ) Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed .. Patch Set 6: (10 comments)

[Impala-ASF-CR] IMPALA-8570: Fix flakiness in test restart statestore query resilience

2019-05-23 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13406 ) Change subject: IMPALA-8570: Fix flakiness in test_restart_statestore_query_resilience .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7290: part 2: Add HS2 support to Impala shell

2019-05-22 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/12884 ) Change subject: IMPALA-7290: part 2: Add HS2 support to Impala shell .. Patch Set 18: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint

2019-05-22 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13299 ) Change subject: IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint .. Patch Set 3: (17 comments)

[Impala-ASF-CR] IMPALA-8538 (part 1) Copied THttp(Server|Transport) from thrift-0.9.3

2019-05-22 Thread Thomas Marshall (Code Review)
Hello Michael Ho, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13298 to look at the new patch set (#2). Change subject: IMPALA-8538 (part 1) Copied THttp(Server|Transport) from thrift-0.9.3

[Impala-ASF-CR] IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based thrift server endpoint

2019-05-22 Thread Thomas Marshall (Code Review)
Hello Bharath Vissapragada, Michael Ho, Sudhanshu Arora, Mike Yoder, Todd Lipcon, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13299 to look at the new patch set (#3). Change subject: IMPALA-8538: HS2 + HTTP(S) + BASIC/LDAP based

[native-toolchain-CR] Patch Thrift to 0.9.3-p6 to eliminate erroneous ssl warnings.

2019-05-22 Thread Thomas Marshall (Code Review)
Thomas Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13383 ) Change subject: Patch Thrift to 0.9.3-p6 to eliminate erroneous ssl warnings. .. Patch Thrift to 0.9.3-p6 to eliminate

[native-toolchain-CR] Patch Thrift to 0.9.3-p6 to eliminate erroneous ssl warnings.

2019-05-22 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13383 ) Change subject: Patch Thrift to 0.9.3-p6 to eliminate erroneous ssl warnings. .. Patch Set 1: Verified+1 -- To view, visit

[native-toolchain-CR] Patch Thrift to 0.9.3-p6 to eliminate erroneous ssl warnings.

2019-05-21 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13383 ) Change subject: Patch Thrift to 0.9.3-p6 to eliminate erroneous ssl warnings. .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7290: part 2: Add HS2 support to Impala shell

2019-05-20 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/12884 ) Change subject: IMPALA-7290: part 2: Add HS2 support to Impala shell .. Patch Set 16: (1 comment)

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-20 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13306 ) Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-20 Thread Thomas Marshall (Code Review)
Hello Michael Ho, Andrew Sherman, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13306 to look at the new patch set (#5). Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed

[Impala-ASF-CR] IMPALA-7290: part 2: Add HS2 support to Impala shell

2019-05-20 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/12884 ) Change subject: IMPALA-7290: part 2: Add HS2 support to Impala shell .. Patch Set 16: (5 comments)

[Impala-ASF-CR] IMPALA-1653: Don't close hiveserver2 session when connection is closed

2019-05-17 Thread Thomas Marshall (Code Review)
Hello Michael Ho, Andrew Sherman, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13306 to look at the new patch set (#4). Change subject: IMPALA-1653: Don't close hiveserver2 session when connection is closed

[Impala-ASF-CR] IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration

2019-05-17 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/13318 ) Change subject: IMPALA-8504: Support CREATE TABLE statement with Kudu/HMS integration .. Patch Set 2: (7 comments)

  1   2   3   4   5   6   >