[kudu-CR] [tools] Kudu table schema in Avro format revisited This is a follow-up patch to 55cab44 addressing the additional comments posted after the change has been cherry picked.

2022-01-13 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18148 ) Change subject: [tools] Kudu table schema in Avro format revisited This is a follow-up patch to 55cab44 addressing the additional comments posted after the change has been cherry picked.

[kudu-CR] [tools] Kudu table schema in Avro format

2022-01-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18121 ) Change subject: [tools] Kudu table schema in Avro format .. [tools] Kudu table schema in Avro format This patch introduces a

[kudu-CR] [tools] Kudu table schema in Avro format

2022-01-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18121 ) Change subject: [tools] Kudu table schema in Avro format .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] [tools] Kudu table schema in Avro format

2022-01-10 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18121 ) Change subject: [tools] Kudu table schema in Avro format .. Patch Set 3: (1 comment)

[kudu-CR] [tools] Kudu table schema in Avro format

2022-01-10 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18121 ) Change subject: [tools] Kudu table schema in Avro format .. Patch Set 2: (5 comments) Mostly minor comments.

[kudu-CR] WIP: Kudu table schema in Avro format

2022-01-05 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18121 ) Change subject: WIP: Kudu table schema in Avro format .. Patch Set 1: (3 comments)

[kudu-CR] [thirdparty] Add jwt-cpp to thirdparty build

2021-12-15 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18103 Change subject: [thirdparty] Add jwt-cpp to thirdparty build .. [thirdparty] Add jwt-cpp to thirdparty build This is a header only

[kudu-CR] [tool] KUDU-3343 Display the "Data state" and "Last status" in remote replica list command.

2021-12-14 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18092 ) Change subject: [tool] KUDU-3343 Display the "Data state" and "Last status" in remote_replica list command. .. Patch Set 6:

[kudu-CR] [consensus] force fsync metadata on XFS

2021-12-14 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18101 ) Change subject: [consensus] force fsync metadata on XFS .. Patch Set 2: Code-Review+1 -- To view, visit

[kudu-CR](branch-1.13.x) [java] bump log4j up to 2.15.0 version

2021-12-13 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18089 ) Change subject: [java] bump log4j up to 2.15.0 version .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-3339 [tool] CLI to add a column

2021-12-13 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17851 ) Change subject: KUDU-3339 [tool] CLI to add a column .. Patch Set 9: Code-Review+1 (1 comment)

[kudu-CR] [tool] KUDU-3343 Display the "Data state" and "Last statue" in remote replica list command.

2021-12-13 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18092 ) Change subject: [tool] KUDU-3343 Display the "Data state" and "Last statue" in remote_replica list command. .. Patch Set 3: (2

[kudu-CR] [kudu-client] Shade stumbleupon async jar to kudu-client

2021-12-13 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has abandoned this change. ( http://gerrit.cloudera.org:8080/18070 ) Change subject: [kudu-client] Shade stumbleupon async jar to kudu-client .. Abandoned Not needed. -- To view, visit

[kudu-CR] [kudu-client] Shade stumbleupon async jar to kudu-client

2021-12-06 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/18070 ) Change subject: [kudu-client] Shade stumbleupon async jar to kudu-client .. Patch Set 1: (1 comment)

[kudu-CR] [java] Enable zip64 for building kudu-subprocess jar

2021-12-05 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has abandoned this change. ( http://gerrit.cloudera.org:8080/18063 ) Change subject: [java] Enable zip64 for building kudu-subprocess jar .. Abandoned Found an alternative to instead exclude elasticsearch and

[kudu-CR] [kudu-client] Shade stumbleupon async jar to kudu-client

2021-12-03 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18070 Change subject: [kudu-client] Shade stumbleupon async jar to kudu-client .. [kudu-client] Shade stumbleupon async jar to kudu-client

[kudu-CR] [java] Enable zip64 for building kudu-subprocess jar

2021-12-01 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18063 Change subject: [java] Enable zip64 for building kudu-subprocess jar .. [java] Enable zip64 for building kudu-subprocess jar One of

[kudu-CR] [tool] CLI to add a column

2021-11-19 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17851 ) Change subject: [tool] CLI to add a column .. Patch Set 4: Code-Review+1 (1 comment)

[kudu-CR] [master] KUDU-3311 Allow to start with diff num of masters

2021-11-03 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17995 ) Change subject: [master] KUDU-3311 Allow to start with diff num of masters .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-3308: Add health check endpoint to webserver

2021-10-28 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17962 ) Change subject: KUDU-3308: Add health check endpoint to webserver .. KUDU-3308: Add health check endpoint to webserver

[kudu-CR] KUDU-3308: Add health check endpoint to webserver

2021-10-28 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17962 ) Change subject: KUDU-3308: Add health check endpoint to webserver .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-3308: Add health check endpoint to webserver

2021-10-27 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17962 ) Change subject: KUDU-3308: Add health check endpoint to webserver .. Patch Set 1: Could you add how did you test this change in the

[kudu-CR] KUDU-3308: Add health check endpoint to webserver

2021-10-27 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17962 ) Change subject: KUDU-3308: Add health check endpoint to webserver .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [client] fix recently introduced ABI compatiblity issue

2021-10-26 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17972 ) Change subject: [client] fix recently introduced ABI compatiblity issue .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [tool] Add a '-list statistics' flag to the 'kudu table list'.

2021-10-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17912 ) Change subject: [tool] Add a '-list_statistics' flag to the 'kudu table list'. .. [tool] Add a '-list_statistics' flag to the

[kudu-CR] KUDU-3330 Define operator<< for MonoTime

2021-10-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17961 ) Change subject: KUDU-3330 Define operator<< for MonoTime .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-3330 Define operator<< for MonoTime

2021-10-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17961 ) Change subject: KUDU-3330 Define operator<< for MonoTime .. Patch Set 4: Code-Review+2 Fix the IWYU issue and ship it! -- To view,

[kudu-CR] KUDU-3308: Add health check endpoint to webserver

2021-10-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17962 ) Change subject: KUDU-3308: Add health check endpoint to webserver .. Patch Set 1: Code-Review+1 Is this API easily extendable to add

[kudu-CR] KUDU-3330 Define operator<< for MonoTime

2021-10-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17961 ) Change subject: KUDU-3330 Define operator<< for MonoTime .. Patch Set 3: (1 comment)

[kudu-CR](gh-pages) [docs] Add info about Informatica-Kudu integration

2021-10-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17941 ) Change subject: [docs] Add info about Informatica-Kudu integration .. Patch Set 6: Code-Review+2 -- To view, visit

[kudu-CR](gh-pages) Fix deprecation error of the --path flag

2021-10-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17940 ) Change subject: Fix deprecation error of the --path flag .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-3297 fix Thrift client used for HMS integration

2021-10-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17958 ) Change subject: KUDU-3297 fix Thrift client used for HMS integration .. Patch Set 1: Code-Review+1 -- To view, visit

[kudu-CR] [docs] suggest installing lsof if running tests

2021-10-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17957 ) Change subject: [docs] suggest installing lsof if running tests .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [tool] Add a '-list statistics' flag to the 'kudu table list'.

2021-10-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17912 ) Change subject: [tool] Add a '-list_statistics' flag to the 'kudu table list'. .. Patch Set 12: Code-Review+2 -- To view, visit

[kudu-CR] [KUDU-3330] Define operator<< for MonoTime

2021-10-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17961 ) Change subject: [KUDU-3330] Define operator<< for MonoTime .. Patch Set 1: (1 comment)

[kudu-CR] [tool] Add a '-list statistics' flag to the 'kudu table list'.

2021-10-20 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17912 ) Change subject: [tool] Add a '-list_statistics' flag to the 'kudu table list'. .. Patch Set 10: (4 comments)

[kudu-CR] [tool] Add a '-list statistics' flag to the 'kudu table list'.

2021-10-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17912 ) Change subject: [tool] Add a '-list_statistics' flag to the 'kudu table list'. .. Patch Set 7: (4 comments)

[kudu-CR] [master] Validate extra config properties for a table

2021-10-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17914 ) Change subject: [master] Validate extra config properties for a table .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [tool] Add a '-list statistics' flag to the 'kudu table list'.

2021-10-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17912 ) Change subject: [tool] Add a '-list_statistics' flag to the 'kudu table list'. .. Patch Set 2: We already have "kudu table

[kudu-CR] [master] Expose error when set an unsupported table extra config

2021-10-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17914 ) Change subject: [master] Expose error when set an unsupported table extra config .. Patch Set 1: (2 comments)

[kudu-CR] [master] KUDU-1885: re-resolve TSDescriptor proxies on network error

2021-10-08 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17909 ) Change subject: [master] KUDU-1885: re-resolve TSDescriptor proxies on network error .. Patch Set 5: Code-Review+1 -- To view,

[kudu-CR] [consensus] KUDU-1620: re-resolve consensus peers on network error

2021-10-04 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17868 ) Change subject: [consensus] KUDU-1620: re-resolve consensus peers on network error .. Patch Set 3: Code-Review+1 -- To view, visit

[kudu-CR] [tool] kudu tool, support add column

2021-09-29 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17851 ) Change subject: [tool] kudu tool, support add column .. Patch Set 2: (6 comments)

[kudu-CR] [consensus] KUDU-1620: re-resolve consensus peers on network error

2021-09-28 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17868 ) Change subject: [consensus] KUDU-1620: re-resolve consensus peers on network error .. Patch Set 2: Code-Review+2 (1 comment)

[kudu-CR] [rpc] Expose KRPC call id to client application

2021-09-23 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17866 ) Change subject: [rpc] Expose KRPC call_id to client application .. Patch Set 1: (1 comment)

[kudu-CR] [master] fix logic error in TableInfo::UpdateMetrics()

2021-09-23 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17856 ) Change subject: [master] fix logic error in TableInfo::UpdateMetrics() .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [master] fix logic error in TableInfo::UpdateMetrics()

2021-09-20 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17856 ) Change subject: [master] fix logic error in TableInfo::UpdateMetrics() .. Patch Set 1: > Patch Set 1: > > ts_tablet_manager.h says

[kudu-CR] [master] fix logic error in TableInfo::UpdateMetrics()

2021-09-20 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17856 ) Change subject: [master] fix logic error in TableInfo::UpdateMetrics() .. Patch Set 1: ts_tablet_manager.h says this. Could you

[kudu-CR] [util] allow DnsResolver to refresh DNS addresses

2021-09-17 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17849 ) Change subject: [util] allow DnsResolver to refresh DNS addresses .. Patch Set 1: Code-Review+2 (1 comment)

[kudu-CR] wip [rpc] KUDU-75: refresh DNS entries if proxies hit a network error

2021-09-14 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17839 ) Change subject: wip [rpc] KUDU-75: refresh DNS entries if proxies hit a network error .. Patch Set 3: (5 comments)

[kudu-CR] [Java] KUDU-3313 Upgrade netty version from 4.1.60.Final to 4.1.65.Final

2021-09-07 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17828 ) Change subject: [Java] KUDU-3313 Upgrade netty version from 4.1.60.Final to 4.1.65.Final .. Patch Set 1: Code-Review+1 -- To view,

[kudu-CR](branch-1.15.x) [docs] update security-related limitations since 1.15.0 release

2021-09-07 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17832 ) Change subject: [docs] update security-related limitations since 1.15.0 release .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [docs] update security-related limitations since 1.15.0 release

2021-08-31 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17823 ) Change subject: [docs] update security-related limitations since 1.15.0 release .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [docs] update security-related limitations since 1.15.0 release

2021-08-31 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17823 ) Change subject: [docs] update security-related limitations since 1.15.0 release .. Patch Set 1: (1 comment)

[kudu-CR] [doc] update list of Kudu design highlights

2021-08-24 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17805 ) Change subject: [doc] update list of Kudu design highlights .. Patch Set 2: (2 comments)

[kudu-CR] [docker] clarify on {MASTER,TSERVER} ARGS

2021-08-24 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17803 ) Change subject: [docker] clarify on {MASTER,TSERVER}_ARGS .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [tool] Support to dump and edit pbc in JSON pretty format

2021-08-24 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17808 ) Change subject: [tool] Support to dump and edit pbc in JSON pretty format .. Patch Set 1: (1 comment)

[kudu-CR] [tablet] small clean-up

2021-08-20 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17797 ) Change subject: [tablet] small clean-up .. Patch Set 2: Code-Review+2 (1 comment)

[kudu-CR] [fuzz-itest] add WAL GCs

2021-08-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17787 ) Change subject: [fuzz-itest] add WAL GCs .. Patch Set 4: (1 comment)

[kudu-CR] [fuzz-itest] add WAL GCs

2021-08-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17787 ) Change subject: [fuzz-itest] add WAL GCs .. Patch Set 1: Code-Review+2 Good addition! -- To view, visit

[kudu-CR] [tools] Add a tool to recover master data from tablet servers

2021-08-11 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/9490 ) Change subject: [tools] Add a tool to recover master data from tablet servers .. Patch Set 10: Code-Review+2 Okay with follow-up

[kudu-CR] [tools] Add a tool to recover master data from tablet servers

2021-08-09 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/9490 ) Change subject: [tools] Add a tool to recover master data from tablet servers .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-3304: [Alter] Support to alter table's replication factor

2021-08-05 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17674 ) Change subject: KUDU-3304: [Alter] Support to alter table's replication factor .. Patch Set 10: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-3304: [Alter] Support to alter table's replication factor

2021-08-03 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17674 ) Change subject: KUDU-3304: [Alter] Support to alter table's replication factor .. Patch Set 8: (1 comment)

[kudu-CR] [tools] Add a tool to recover master data from tablet servers

2021-08-02 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/9490 ) Change subject: [tools] Add a tool to recover master data from tablet servers .. Patch Set 6: Code-Review+1 (5 comments) LGTM. Minor

[kudu-CR] add http api, get version info

2021-07-31 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17745 ) Change subject: add http api, get version info .. Patch Set 3: (1 comment) Could you add information about testing in the commit

[kudu-CR] [helm] fix usage of multiple directories

2021-07-30 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17739 ) Change subject: [helm] fix usage of multiple directories .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [docker] KUDU-3307: allow using multiple directories

2021-07-30 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17738 ) Change subject: [docker] KUDU-3307: allow using multiple directories .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [helm] fix usage of multiple directories

2021-07-30 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17739 ) Change subject: [helm] fix usage of multiple directories .. Patch Set 1: (2 comments)

[kudu-CR] [docker] KUDU-3307: allow using multiple directories

2021-07-30 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17738 ) Change subject: [docker] KUDU-3307: allow using multiple directories .. Patch Set 2: (2 comments) Thanks for taking this up,

[kudu-CR] [tools] --scan batch size for 'table scan' and 'perf table scan'

2021-07-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17708 ) Change subject: [tools] --scan_batch_size for 'table scan' and 'perf table_scan' .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [tools] --scan batch size for 'table scan' and 'perf table scan'

2021-07-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17708 ) Change subject: [tools] --scan_batch_size for 'table scan' and 'perf table_scan' .. Patch Set 1: (2 comments)

[kudu-CR](branch-1.15.x) KUDU-3297 fix RPC negotiations with cyrus-sasl-gssapi-2.1.27-5 and newer

2021-07-19 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17631 ) Change subject: KUDU-3297 fix RPC negotiations with cyrus-sasl-gssapi-2.1.27-5 and newer .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR](branch-1.15.x) [cmake] Boost to use /dev/[u]random for UUID generation (take 2)

2021-07-19 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17699 ) Change subject: [cmake] Boost to use /dev/[u]random for UUID generation (take 2) .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [test] deflake dynamic multi master-test

2021-07-19 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17692 ) Change subject: [test] deflake dynamic_multi_master-test .. Patch Set 2: (3 comments)

[kudu-CR] [master] make automatic AddMaster resilient to leadership changes

2021-07-19 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17691 ) Change subject: [master] make automatic AddMaster resilient to leadership changes .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [client] actionable error message for KuduSession::Flush()

2021-07-12 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17673 ) Change subject: [client] actionable error message for KuduSession::Flush() .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [master] AddMaster and copy sys catalog automatically

2021-07-06 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17528 ) Change subject: [master] AddMaster and copy sys catalog automatically .. Patch Set 3: (7 comments)

[kudu-CR] [master] allow setting BlockCacheMetrics twice

2021-07-06 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17636 ) Change subject: [master] allow setting BlockCacheMetrics twice .. Patch Set 3: Code-Review+2 (1 comment)

[kudu-CR] [master] allow setting BlockCacheMetrics twice

2021-06-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17636 ) Change subject: [master] allow setting BlockCacheMetrics twice .. Patch Set 1: Code-Review+1 (1 comment)

[kudu-CR] [txn system client] MonoDelta by value in CoordinateTransactionAsync()

2021-06-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17611 ) Change subject: [txn_system_client] MonoDelta by value in CoordinateTransactionAsync() .. Patch Set 2: Sorry I didn't understand.

[kudu-CR] [docs] Updates to the RELEASING doc

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17624 ) Change subject: [docs] Updates to the RELEASING doc .. [docs] Updates to the RELEASING doc Minor issues and clarifications

[kudu-CR] [examples] Bump up Kudu version to 1.15.0 for examples

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17622 ) Change subject: [examples] Bump up Kudu version to 1.15.0 for examples .. [examples] Bump up Kudu version to 1.15.0 for

[kudu-CR] [docs] Add 1.15.0 release notes to previous release notes

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17621 ) Change subject: [docs] Add 1.15.0 release notes to previous release notes .. [docs] Add 1.15.0 release notes to previous

[kudu-CR](branch-1.15.x) [version] Bump up the version to 1.15.1-SNAPSHOT for next release

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17623 ) Change subject: [version] Bump up the version to 1.15.1-SNAPSHOT for next release .. [version] Bump up the version to

[kudu-CR] [docs] Updates to the RELEASING doc

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17624 Change subject: [docs] Updates to the RELEASING doc .. [docs] Updates to the RELEASING doc Minor issues and clarifications to the

[kudu-CR](branch-1.15.x) [version] Bump up the version to 1.15.1-SNAPSHOT for next release

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17623 Change subject: [version] Bump up the version to 1.15.1-SNAPSHOT for next release .. [version] Bump up the version to 1.15.1-SNAPSHOT

[kudu-CR] [docs] Add 1.15.0 release notes to previous release notes

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17621 Change subject: [docs] Add 1.15.0 release notes to previous release notes .. [docs] Add 1.15.0 release notes to previous release notes

[kudu-CR] [examples] Bump up Kudu version to 1.15.0 for examples

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17622 Change subject: [examples] Bump up Kudu version to 1.15.0 for examples .. [examples] Bump up Kudu version to 1.15.0 for examples

[kudu-CR](gh-pages) [blog] Add a blog post for 1.15.0 release

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17617 ) Change subject: [blog] Add a blog post for 1.15.0 release .. [blog] Add a blog post for 1.15.0 release Change-Id:

[kudu-CR](gh-pages) [site] Update website for 1.15.0 release

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17616 ) Change subject: [site] Update website for 1.15.0 release .. [site] Update website for 1.15.0 release Change-Id:

[kudu-CR](gh-pages) [blog] Add a blog post for 1.15.0 release

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17617 ) Change subject: [blog] Add a blog post for 1.15.0 release .. Patch Set 2: (1 comment)

[kudu-CR](gh-pages) [blog] Add a blog post for 1.15.0 release

2021-06-22 Thread Bankim Bhavsar (Code Review)
Hello Alexey Serbin, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/17617 to look at the new patch set (#2). Change subject: [blog] Add a blog post for 1.15.0 release

[kudu-CR](gh-pages) [blog] Add a blog post for 1.15.0 release

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17617 Change subject: [blog] Add a blog post for 1.15.0 release .. [blog] Add a blog post for 1.15.0 release Change-Id:

[kudu-CR](gh-pages) [site] Update website for 1.15.0 release

2021-06-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17616 Change subject: [site] Update website for 1.15.0 release .. [site] Update website for 1.15.0 release Change-Id:

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

2021-06-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has abandoned this change. ( http://gerrit.cloudera.org:8080/17613 ) Change subject: [docs] Add 1.15.0 release notes to prior release notes .. Abandoned This meant for master branch only. -- To view, visit

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

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

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

2021-06-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17613 Change subject: [docs] Add 1.15.0 release notes to prior release notes .. [docs] Add 1.15.0 release notes to prior release notes

[kudu-CR] [metrics] KUDU-3269: Add UUID of the server into the metrics output

2021-06-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17607 ) Change subject: [metrics] KUDU-3269: Add UUID of the server into the metrics output .. Patch Set 1: (1 comment)

[kudu-CR] [fs] KUDU-3161: Include FileSystem Path in UUID Mismatch Error

2021-06-21 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17606 ) Change subject: [fs] KUDU-3161: Include FileSystem Path in UUID Mismatch Error .. Patch Set 1: Code-Review+2 -- To view, visit

  1   2   3   4   5   6   7   8   9   10   >