[kudu-CR] [test] check C++ client API for C++98 compatibility

2021-06-19 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17603 ) Change subject: [test] check C++ client API for C++98 compatibility .. Patch Set 1: Code-Review+2 Is this script/check run during

[kudu-CR] [docker] Add the license and notice file to the images

2021-06-14 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17594 ) Change subject: [docker] Add the license and notice file to the images .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [clock] use GTEST SKIP in BuiltinNtpWithMiniChronydTest

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17587 ) Change subject: [clock] use GTEST_SKIP in BuiltinNtpWithMiniChronydTest .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 4: (2 comments)

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17584 ) Change subject: [util] flag to specify hosts to fail DNS resolution .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [consensus] KUDU-2302: don't crash if new leader can't resolve peer

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17585 ) Change subject: [consensus] KUDU-2302: don't crash if new leader can't resolve peer .. Patch Set 2: (2 comments)

[kudu-CR] [util] flag to specify hosts to fail DNS resolution

2021-06-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17584 ) Change subject: [util] flag to specify hosts to fail DNS resolution .. Patch Set 1: (1 comment)

[kudu-CR](branch-1.15.x) docs: use PNG instead of SVG for image

2021-06-10 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17579 ) Change subject: docs: use PNG instead of SVG for image .. docs: use PNG instead of SVG for image The SVG resulted in a

[kudu-CR](branch-1.15.x) [docker] Add support for publishing UBI based runtime images

2021-06-10 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17573 ) Change subject: [docker] Add support for publishing UBI based runtime images .. [docker] Add support for publishing UBI based

[kudu-CR](branch-1.15.x) [docker] Add support for publishing UBI based runtime images

2021-06-10 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17573 ) Change subject: [docker] Add support for publishing UBI based runtime images .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) docs: use PNG instead of SVG for image

2021-06-10 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17579 Change subject: docs: use PNG instead of SVG for image .. docs: use PNG instead of SVG for image The SVG resulted in a failed RAT

[kudu-CR] docs: use PNG instead of SVG for image

2021-06-10 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17578 ) Change subject: docs: use PNG instead of SVG for image .. docs: use PNG instead of SVG for image The SVG resulted in a

[kudu-CR] docs: use PNG instead of SVG for image

2021-06-10 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17578 ) Change subject: docs: use PNG instead of SVG for image .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [docker] Add option to force the latest tag

2021-06-10 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17574 ) Change subject: [docker] Add option to force the latest tag .. Patch Set 1: Verified+1 (2 comments)

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-09 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17539 ) Change subject: [docs] Add release notes for 1.15.0 release .. [docs] Add release notes for 1.15.0 release The rendered

[kudu-CR](branch-1.15.x) [test] make new scenario in security-itest more robust

2021-06-08 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17562 ) Change subject: [test] make new scenario in security-itest more robust .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) [client] replace Equals() with operator==()

2021-06-08 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17563 ) Change subject: [client] replace Equals() with operator==() .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] consensus: make some methods void

2021-06-08 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17565 ) Change subject: consensus: make some methods void .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [docker] Add support for publishing UBI based runtime images

2021-06-08 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17564 ) Change subject: [docker] Add support for publishing UBI based runtime images .. Patch Set 1: Code-Review+1 -- To view, visit

[kudu-CR](branch-1.15.x) KUDU-3291: properly disambiguate between deltas of a row with the same timestamp

2021-06-08 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17561 ) Change subject: KUDU-3291: properly disambiguate between deltas of a row with the same timestamp .. Patch Set 1: Code-Review+2 --

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-08 Thread Bankim Bhavsar (Code Review)
Hello Mahesh Reddy, Alexey Serbin, shenxingwuy...@gmail.com, Hongjiang Zhang, Attila Bukor, Yifan Zhang, Kudu Jenkins, Andrew Wong, Abhishek Chennaka, Grant Henke, Li Zhiming, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17539 to look at the new patch

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-08 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17539 ) Change subject: [docs] Add release notes for 1.15.0 release .. Patch Set 7: (1 comment)

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-08 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17539 ) Change subject: [docs] Add release notes for 1.15.0 release .. Patch Set 6: (1 comment)

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-08 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17539 ) Change subject: [docs] Add release notes for 1.15.0 release .. Patch Set 6: > Patch Set 5: > > (1 comment) > > Although not too

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-08 Thread Bankim Bhavsar (Code Review)
Hello Mahesh Reddy, Alexey Serbin, shenxingwuy...@gmail.com, Hongjiang Zhang, Attila Bukor, Yifan Zhang, Kudu Jenkins, Andrew Wong, Abhishek Chennaka, Grant Henke, Li Zhiming, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17539 to look at the new patch

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-05 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17539 ) Change subject: [docs] Add release notes for 1.15.0 release .. Patch Set 5: (3 comments)

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-05 Thread Bankim Bhavsar (Code Review)
Hello Mahesh Reddy, Alexey Serbin, shenxingwuy...@gmail.com, Hongjiang Zhang, Attila Bukor, Yifan Zhang, Kudu Jenkins, Andrew Wong, Abhishek Chennaka, Grant Henke, Li Zhiming, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17539 to look at the new patch

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-04 Thread Bankim Bhavsar (Code Review)
Hello Mahesh Reddy, Alexey Serbin, shenxingwuy...@gmail.com, Hongjiang Zhang, Attila Bukor, Yifan Zhang, Kudu Jenkins, Andrew Wong, Abhishek Chennaka, Grant Henke, Li Zhiming, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17539 to look at the new patch

[kudu-CR] [docs] Add release notes for 1.15.0 release

2021-06-04 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has abandoned this change. ( http://gerrit.cloudera.org:8080/17543 ) Change subject: [docs] Add release notes for 1.15.0 release .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/17543 To

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-04 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17539 ) Change subject: [docs] Add release notes for 1.15.0 release .. Patch Set 3: (2 comments)

[kudu-CR] [docs] Add release notes for 1.15.0 release

2021-06-04 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17543 Change subject: [docs] Add release notes for 1.15.0 release .. [docs] Add release notes for 1.15.0 release The rendered version can

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-04 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17539 ) Change subject: [docs] Add release notes for 1.15.0 release .. Patch Set 3: (1 comment)

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-04 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17539 ) Change subject: [docs] Add release notes for 1.15.0 release .. Patch Set 3: (6 comments)

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-04 Thread Bankim Bhavsar (Code Review)
Hello Mahesh Reddy, Alexey Serbin, shenxingwuy...@gmail.com, Hongjiang Zhang, Attila Bukor, Yifan Zhang, Kudu Jenkins, Andrew Wong, Abhishek Chennaka, Grant Henke, Li Zhiming, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17539 to look at the new patch

[kudu-CR](branch-1.15.x) KUDU-3265 txn-related options to 'kudu perf loadgen'

2021-06-04 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17540 ) Change subject: KUDU-3265 txn-related options to 'kudu perf loadgen' .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-03 Thread Bankim Bhavsar (Code Review)
Hello Mahesh Reddy, Alexey Serbin, shenxingwuy...@gmail.com, Hongjiang Zhang, Attila Bukor, Yifan Zhang, Kudu Jenkins, Andrew Wong, Abhishek Chennaka, Grant Henke, Li Zhiming, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17539 to look at the new patch

[kudu-CR](branch-1.15.x) [docs] Add release notes for 1.15.0 release

2021-06-03 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17539 Change subject: [docs] Add release notes for 1.15.0 release .. [docs] Add release notes for 1.15.0 release The rendered version can

[kudu-CR](branch-1.15.x) [docs] add a design overview for multi-row transactions

2021-06-03 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17536 ) Change subject: [docs] add a design overview for multi-row transactions .. [docs] add a design overview for multi-row

[kudu-CR](branch-1.15.x) [hms] Avoid blocking table alterations that are not relevant to Kudu

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17537 ) Change subject: [hms] Avoid blocking table alterations that are not relevant to Kudu .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR](branch-1.15.x) [docs] add a design overview for multi-row transactions

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17536 ) Change subject: [docs] add a design overview for multi-row transactions .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) [java] KUDU-3267 Improve logging on writes to non-existent partitions

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17535 ) Change subject: [java] KUDU-3267 Improve logging on writes to non-existent partitions .. [java] KUDU-3267 Improve logging on

[kudu-CR](branch-1.15.x) [consensus] --evict failed followers is a runtime flag

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17534 ) Change subject: [consensus] --evict_failed_followers is a runtime flag .. [consensus] --evict_failed_followers is a runtime

[kudu-CR](branch-1.15.x) [consensus] --evict failed followers is a runtime flag

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has removed a vote on this change. Change subject: [consensus] --evict_failed_followers is a runtime flag .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/17534 To

[kudu-CR](branch-1.15.x) [consensus] --evict failed followers is a runtime flag

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17534 ) Change subject: [consensus] --evict_failed_followers is a runtime flag .. Patch Set 1: Verified+1 Unrelated test failure. -- To

[kudu-CR](branch-1.15.x) [java] KUDU-3267 Improve logging on writes to non-existent partitions

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17535 ) Change subject: [java] KUDU-3267 Improve logging on writes to non-existent partitions .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR](branch-1.15.x) [consensus] --evict failed followers is a runtime flag

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17534 ) Change subject: [consensus] --evict_failed_followers is a runtime flag .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) KUDU-2612 allow system user to read list of table replicas

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17532 ) Change subject: KUDU-2612 allow system user to read list of table replicas .. KUDU-2612 allow system user to read list of

[kudu-CR](branch-1.15.x) KUDU-2612 allow system user to read list of table replicas

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17532 ) Change subject: KUDU-2612 allow system user to read list of table replicas .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) KUDU-2612 allow system user to read list of table replicas

2021-06-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17532 ) Change subject: KUDU-2612 allow system user to read list of table replicas .. Patch Set 1: > Patch Set 1: Verified-1 > > Build

[kudu-CR] [consensus] --evict failed followers is a runtime flag

2021-05-27 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17522 ) Change subject: [consensus] --evict_failed_followers is a runtime flag .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) Bump up version to 1.15.0

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17519 ) Change subject: Bump up version to 1.15.0 .. Bump up version to 1.15.0 Change-Id: I6eb43b356102675b432a7ba879df00e1d00376b7

[kudu-CR](branch-1.15.x) Bump up version to 1.15.0

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has removed a vote on this change. Change subject: Bump up version to 1.15.0 .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/17519 To unsubscribe, visit

[kudu-CR](branch-1.15.x) Bump up version to 1.15.0

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17519 ) Change subject: Bump up version to 1.15.0 .. Patch Set 1: Verified+1 > Patch Set 1: Verified-1 > > Build Failed > >

[kudu-CR](branch-1.15.x) Bump up version to 1.15.0

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17519 Change subject: Bump up version to 1.15.0 .. Bump up version to 1.15.0 Change-Id: I6eb43b356102675b432a7ba879df00e1d00376b7 --- M

[kudu-CR](branch-1.15.x) [txn manager] allow a super user to call txn-related RPCs

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17517 ) Change subject: [txn_manager] allow a super user to call txn-related RPCs .. [txn_manager] allow a super user to call

[kudu-CR](branch-1.15.x) [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17516 ) Change subject: [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag .. [master] KUDU-2181 Fix

[kudu-CR](branch-1.15.x) [txn manager] allow a super user to call txn-related RPCs

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17517 ) Change subject: [txn_manager] allow a super user to call txn-related RPCs .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag

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

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

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17500 ) Change subject: [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag .. [master] KUDU-2181 Fix

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

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17500 ) Change subject: [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag .. Patch Set 2: (1 comment)

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

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has removed a vote on this change. Change subject: [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

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

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17500 ) Change subject: [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag .. Patch Set 2: Verified+1 > Patch Set

[kudu-CR](branch-1.15.x) [tests] TsRecoveryITestDeathTest isn't for for TSAN buidls

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17511 ) Change subject: [tests] TsRecoveryITestDeathTest isn't for for TSAN buidls .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) [backup] KUDU-3183 Add --newDatabaseName and --removeImpalaPrefix options to restore job

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17513 ) Change subject: [backup] KUDU-3183 Add --newDatabaseName and --removeImpalaPrefix options to restore job .. Patch Set 1:

[kudu-CR](branch-1.15.x) [tests] TsRecoveryITestDeathTest isn't for for TSAN buidls

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17511 ) Change subject: [tests] TsRecoveryITestDeathTest isn't for for TSAN buidls .. Patch Set 1: > Patch Set 1: Verified-1 > > Build

[kudu-CR](branch-1.15.x) [consensus] a small optimisation on log prefix

2021-05-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17512 ) Change subject: [consensus] a small optimisation on log prefix .. Patch Set 1: Code-Review+2 -- To view, visit

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

2021-05-26 Thread Bankim Bhavsar (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17500 to look at the new patch set (#2). Change subject: [master] KUDU-2181 Fix duplicate master address and remove unsafe flag tag

[kudu-CR] [test] minor clean up on TestUnsafeChangeConfigLeaderWithPendingConfig

2021-05-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17504 ) Change subject: [test] minor clean up on TestUnsafeChangeConfigLeaderWithPendingConfig .. Patch Set 1: > Patch Set 1: > > > > Patch

[kudu-CR] [test] minor clean up on TestUnsafeChangeConfigLeaderWithPendingConfig

2021-05-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17504 ) Change subject: [test] minor clean up on TestUnsafeChangeConfigLeaderWithPendingConfig .. Patch Set 1: > Patch Set 1: > > > Why

[kudu-CR] [test] minor clean up on TestUnsafeChangeConfigLeaderWithPendingConfig

2021-05-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17504 ) Change subject: [test] minor clean up on TestUnsafeChangeConfigLeaderWithPendingConfig .. Patch Set 1: Why remove those info log

[kudu-CR](branch-1.15.x) KUDU-1884: set sasl protocol name for the TxnSystemClient

2021-05-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17509 ) Change subject: KUDU-1884: set sasl_protocol_name for the TxnSystemClient .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) KUDU-2612: disable TxnSystemClient initialization by default

2021-05-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17506 ) Change subject: KUDU-2612: disable TxnSystemClient initialization by default .. Patch Set 1: Code-Review+2 -- To view, visit

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

[kudu-CR](branch-1.15.x) KUDU-3223: management of per-table limit by kudu CLI

2021-05-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17481 ) Change subject: KUDU-3223: management of per-table limit by kudu CLI .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [java] a property to show output while running a test

2021-05-20 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17479 ) Change subject: [java] a property to show output while running a test .. Patch Set 2: Code-Review+1 This looks very useful. +1

[kudu-CR](branch-1.15.x) [txns] add fields for create and update times

2021-05-19 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17476 ) Change subject: [txns] add fields for create and update times .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [kserver] output info on server-wide size limit

2021-05-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17474 ) Change subject: [kserver] output info on server-wide size limit .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) KUDU-2612 automatically flush sessions on txn commit (Java client)

2021-05-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17471 ) Change subject: KUDU-2612 automatically flush sessions on txn commit (Java client) .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [kserver] output info on server-wide size limit

2021-05-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17474 ) Change subject: [kserver] output info on server-wide size limit .. Patch Set 1: Code-Review+2 (1 comment)

[kudu-CR](branch-1.15.x) [util] Add special handling for empty strings in FastHash

2021-05-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17470 ) Change subject: [util] Add special handling for empty strings in FastHash .. [util] Add special handling for empty strings in

[kudu-CR](branch-1.15.x) [util] Add special handling for empty strings in FastHash

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has removed a vote on this change. Change subject: [util] Add special handling for empty strings in FastHash .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/17470 To

[kudu-CR](branch-1.15.x) [util] Add special handling for empty strings in FastHash

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17470 ) Change subject: [util] Add special handling for empty strings in FastHash .. Patch Set 1: Verified+1 Unrelated test failure in

[kudu-CR](branch-1.15.x) [txns] add a means to get transaction metadata from participant

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17467 ) Change subject: [txns] add a means to get transaction metadata from participant .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) [txns][tools] show details of a transaction

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17468 ) Change subject: [txns][tools] show details of a transaction .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) [docs][site] Add a note about Kudu version to doc pages.

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17456 ) Change subject: [docs][site] Add a note about Kudu version to doc pages. .. [docs][site] Add a note about Kudu version to doc

[kudu-CR](branch-1.15.x) [util] Add special handling for empty strings in FastHash

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17470 Change subject: [util] Add special handling for empty strings in FastHash .. [util] Add special handling for empty strings in FastHash

[kudu-CR] [util] Add special handling for empty strings in FastHash

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17376 ) Change subject: [util] Add special handling for empty strings in FastHash .. [util] Add special handling for empty strings in

[kudu-CR] [util] Add special handling for empty strings in FastHash

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17376 ) Change subject: [util] Add special handling for empty strings in FastHash .. Patch Set 5: (1 comment)

[kudu-CR] Update version to 1.16.0-SNAPSHOT

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17449 ) Change subject: Update version to 1.16.0-SNAPSHOT .. Update version to 1.16.0-SNAPSHOT Change-Id:

[kudu-CR] [util] Add special handling for empty strings in FastHash

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17376 ) Change subject: [util] Add special handling for empty strings in FastHash .. Patch Set 5: (1 comment)

[kudu-CR](branch-1.15.x) KUDU-2612 register txn Java sessions with client

2021-05-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17453 ) Change subject: KUDU-2612 register txn Java sessions with client .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.15.x) [docs][site] Add a note about Kudu version to doc pages.

2021-05-15 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17456 Change subject: [docs][site] Add a note about Kudu version to doc pages. .. [docs][site] Add a note about Kudu version to doc pages.

[kudu-CR](gh-pages) [docs][site] Add links to documentation on the releases index page

2021-05-15 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17439 ) Change subject: [docs][site] Add links to documentation on the releases index page .. [docs][site] Add links to

[kudu-CR](gh-pages) [docs][site] Add links to documentation on the releases index page

2021-05-15 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17439 ) Change subject: [docs][site] Add links to documentation on the releases index page .. Patch Set 1: Verified+1 -- To view, visit

[kudu-CR] [docs][site] Add a note about Kudu version to doc pages.

2021-05-15 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17438 ) Change subject: [docs][site] Add a note about Kudu version to doc pages. .. [docs][site] Add a note about Kudu version to doc

<    1   2   3   4   5   6   7   8   9   10   >