[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:

[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)

[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)

[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

[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

[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

[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

[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:

[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 > >

[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

[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 ..

[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

[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

[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

[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

[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 ..

[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

[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

[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 > >

[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

[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

[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

[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

[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: *

[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

[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: *

[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

[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

[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

[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

[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

[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

[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

[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:

[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

[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