[kudu-CR] KUDU-2612: disable TxnSystemClient initialization by default

2021-05-24 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17499 to look at the new patch set (#3). Change subject: KUDU-2612: disable TxnSystemClient initialization by default ...

[kudu-CR](branch-1.15.x) [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17498 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Patch Set 2: Verified+1 Unrelated test failure

[kudu-CR](branch-1.15.x) [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/1

[kudu-CR](branch-1.15.x) [kserver] output info on server-wide size limit

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17495 ) Change subject: [kserver] output info on server-wide size limit .. Patch Set 1: Code-Review+2 Added +2, but I guess Bankim's +2 is nee

[kudu-CR] KUDU-2612: disable TxnSystemClient initialization by default

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17499 ) Change subject: KUDU-2612: disable TxnSystemClient initialization by default .. Patch Set 2: Code-Review+1 (1 comment) Overall looks

[kudu-CR] KUDU-1884: set sasl protocol name for the TxnSystemClient

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#5) to the change originally created by Grant Henke. ( http://gerrit.cloudera.org:8080/17480 ) Change subject: KUDU-1884: set sasl_protocol_name for the TxnSystemClient .. KUDU-1884: s

[kudu-CR] KUDU-2612: disable TxnSystemClient initialization by default

2021-05-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17499 ) Change subject: KUDU-2612: disable TxnSystemClient initialization by default .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:808

[kudu-CR] KUDU-2612: disable TxnSystemClient initialization by default

2021-05-24 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17499 to look at the new patch set (#2). Change subject: KUDU-2612: disable TxnSystemClient initialization by default .

[kudu-CR] KUDU-2612: disable TxnSystemClient initialization by default

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17499 ) Change subject: KUDU-2612: disable TxnSystemClient initialization by default .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8

[kudu-CR] WIP [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag

2021-05-24 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17500 Change subject: WIP [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag .. WIP [master] KUDU-2181 Fix duplicate

[kudu-CR] KUDU-2612: disable TxnSystemClient initialization by default

2021-05-24 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17499 Change subject: KUDU-2612: disable TxnSystemClient initialization by default .. KUDU-2612: disable TxnSystemClient initialization by defau

[kudu-CR] [backup] KUDU-3183 Add tablePrefix option to the Kudu restore job   While Kudu does not have a notion of database, usually the full table name is stored as . on kudu sid

2021-05-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17388 ) Change subject: [backup] KUDU-3183 Add tablePrefix option to the Kudu restore job   While Kudu does not have a notion of database, usually the full table name is stored as . on kudu side. (NOTE: database n

[kudu-CR] [backup] KUDU-3183 Add tablePrefix option to the Kudu restore job   While Kudu does not have a notion of database, usually the full table name is stored as . on kudu sid

2021-05-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17388 ) Change subject: [backup] KUDU-3183 Add tablePrefix option to the Kudu restore job   While Kudu does not have a notion of database, usually the full table name is stored as . on kudu side. (NOTE: database n

[kudu-CR] WIP: KUDU-1884: Set sasl protocol name for the TxnSystemClient

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#4) to the change originally created by Grant Henke. ( http://gerrit.cloudera.org:8080/17480 ) Change subject: WIP: KUDU-1884: Set sasl_protocol_name for the TxnSystemClient .. WIP: KU

[kudu-CR](branch-1.15.x) [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17498 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Patch Set 2: > Build Failed > > http://jenkin

[kudu-CR](branch-1.15.x) [kserver] output info on server-wide size limit

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17495 ) Change subject: [kserver] output info on server-wide size limit .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.clouder

[kudu-CR](branch-1.15.x) [kserver] output info on server-wide size limit

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17495 ) Change subject: [kserver] output info on server-wide size limit .. Patch Set 1: > > Patch Set 1: Verified-1 > > > > Build Failed >

[kudu-CR](branch-1.15.x) [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#2) to the change originally created by Grant Henke. ( http://gerrit.cloudera.org:8080/17498 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. [t

[kudu-CR](branch-1.15.x) [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17498 Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. [txns] Allow the TxnSystemClient to run in Authoriz

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17490 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. [txns] Allow the TxnSystemClient to run

[kudu-CR](branch-1.15.x) [kserver] output info on server-wide size limit

2021-05-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17495 ) Change subject: [kserver] output info on server-wide size limit .. Patch Set 1: yes, this is a clean backport which shouldn't introduce

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17490 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Patch Set 4: Code-Review+2 -- To view, visit htt

[kudu-CR](branch-1.15.x) [security] fix const-correctness of some ACL methods

2021-05-24 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17497 ) Change subject: [security] fix const-correctness of some ACL methods .. [security] fix const-correctness of some ACL methods Thi

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/1

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17490 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Patch Set 4: Verified+1 unrelated test failure

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#4) to the change originally created by Grant Henke. ( http://gerrit.cloudera.org:8080/17490 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. [t

[kudu-CR](branch-1.15.x) [java] a property to show output while running a test

2021-05-24 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17494 ) Change subject: [java] a property to show output while running a test .. [java] a property to show output while running a test W

[kudu-CR](branch-1.15.x) [security] fix const-correctness of some ACL methods

2021-05-24 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17497 ) Change subject: [security] fix const-correctness of some ACL methods .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.c

[kudu-CR](branch-1.15.x) [kserver] output info on server-wide size limit

2021-05-24 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17495 ) Change subject: [kserver] output info on server-wide size limit .. Patch Set 1: > Patch Set 1: Verified-1 > > Build Failed > > http:/

[kudu-CR](branch-1.15.x) [java] a property to show output while running a test

2021-05-24 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17494 ) Change subject: [java] a property to show output while running a test .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.

[kudu-CR](branch-1.15.x) [security] fix const-correctness of some ACL methods

2021-05-24 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17497 Change subject: [security] fix const-correctness of some ACL methods .. [security] fix const-correctness of some ACL methods This patch f

[kudu-CR] [security] fix const-correctness of some ACL methods

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17496 ) Change subject: [security] fix const-correctness of some ACL methods .. [security] fix const-correctness of some ACL methods T

[kudu-CR] [security] fix const-correctness of some ACL methods

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [security] fix const-correctness of some ACL methods .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/17496 To unsubsc

[kudu-CR] [security] fix const-correctness of some ACL methods

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17496 ) Change subject: [security] fix const-correctness of some ACL methods .. Patch Set 1: Verified+1 unrelated test failure: * ToolTest.T

[kudu-CR] [tests] TsRecoveryITestDeathTest isn't for for TSAN buidls

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [tests] TsRecoveryITestDeathTest isn't for for TSAN buidls .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/17491 To u

[kudu-CR] [tests] TsRecoveryITestDeathTest isn't for for TSAN buidls

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17491 ) Change subject: [tests] TsRecoveryITestDeathTest isn't for for TSAN buidls .. Patch Set 1: Verified+1 unrelated test failures: * Txn

[kudu-CR] [security] fix const-correctness of some ACL methods

2021-05-24 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17496 ) Change subject: [security] fix const-correctness of some ACL methods .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.c

[kudu-CR] [security] fix const-correctness of some ACL methods

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17496 Change subject: [security] fix const-correctness of some ACL methods .. [security] fix const-correctness of some ACL methods This patch

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17490 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Patch Set 3: > Thank you for fixing this. > >

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17490 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Patch Set 3: Looks like I need to fix up a few te

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17490 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Patch Set 3: Code-Review+2 Thank you for fixing

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17490 ) Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments .. Patch Set 3: Code-Review+2 Nice find! -- To vie

[kudu-CR](branch-1.15.x) [kserver] output info on server-wide size limit

2021-05-24 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17495 Change subject: [kserver] output info on server-wide size limit .. [kserver] output info on server-wide size limit With this patch, Kudu

[kudu-CR](branch-1.15.x) [java] a property to show output while running a test

2021-05-24 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17494 Change subject: [java] a property to show output while running a test .. [java] a property to show output while running a test While work

[kudu-CR] [txns] Allow the TxnSystemClient to run in Authorized environments

2021-05-24 Thread Grant Henke (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17490 to look at the new patch set (#3). Change subject: [txns] Allow the TxnSystemClient to run in Authorized environments ...

[kudu-CR](branch-1.15.x) KUDU-3277 fix concurrent session flush issue in Java client

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17493 ) Change subject: KUDU-3277 fix concurrent session flush issue in Java client .. KUDU-3277 fix concurrent session flush issue in

[kudu-CR](branch-1.15.x) [java] utility to detect sanitizer type used for kudu CLI

2021-05-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17492 ) Change subject: [java] utility to detect sanitizer type used for kudu CLI .. [java] utility to detect sanitizer type used for k

[kudu-CR] [txns] Authorization error when running transactional workload in secure tests

2021-05-24 Thread Grant Henke (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17490 to look at the new patch set (#2). Change subject: [txns] Authorization error when running transactional workload in secure tests ...

[kudu-CR] [backup] KUDU-3183 Add tablePrefix option to the Kudu restore job   While Kudu does not have a notion of database, usually the full table name is stored as . on kudu sid

2021-05-24 Thread Abhishek Chennaka (Code Review)
Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/17388 ) Change subject: [backup] KUDU-3183 Add tablePrefix option to the Kudu restore job   While Kudu does not have a notion of database, usually the full table name is stored as . on kudu side. (NOTE: data

[kudu-CR] [backup] KUDU-3183 Add tablePrefix option to the Kudu restore job   While Kudu does not have a notion of database, usually the full table name is stored as . on kudu sid

2021-05-24 Thread Abhishek Chennaka (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17388 to look at the new patch set (#6). Change subject: [backup] KUDU-3183 Add tablePrefix option to the Kudu restore job   While Kudu does not hav

[kudu-CR](branch-1.15.x) KUDU-3277 fix concurrent session flush issue in Java client

2021-05-24 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17493 ) Change subject: KUDU-3277 fix concurrent session flush issue in Java client .. Patch Set 1: Code-Review+2 -- To view, visit http://g