[kudu-CR] (wip) KUDU-2612: restrict TxnStatusManager calls to be made by the leader only

2021-01-11 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16648 to look at the new patch set (#5). Change subject: (wip) KUDU-2612: restrict TxnStatusManager calls to be made by the leader only

[kudu-CR] [java] Add JAAS example to client API doc

2021-01-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16940 ) Change subject: [java] Add JAAS example to client API doc .. Patch Set 1: (2 comments)

[kudu-CR] [master] KUDU-2181 Raft ChangeConfig request to remove a master

2021-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16936 ) Change subject: [master] KUDU-2181 Raft ChangeConfig request to remove a master .. Patch Set 1: (10 comments)

[kudu-CR] KUDU-3205: Fix building scan tokens when tablet not found errors occur

2021-01-11 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16941 to look at the new patch set (#2). Change subject: KUDU-3205: Fix building scan tokens when tablet not found errors occur

[kudu-CR] [util] add a few new metrics in MaintenanceManager

2021-01-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16938 ) Change subject: [util] add a few new metrics in MaintenanceManager .. Patch Set 2: (3 comments)

[kudu-CR] [java] Add JAAS example to client API doc

2021-01-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16940 ) Change subject: [java] Add JAAS example to client API doc .. Patch Set 1: (3 comments) Thanks for putting this change together!

[kudu-CR] [master] KUDU-2181 Raft ChangeConfig request to remove a master

2021-01-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16936 ) Change subject: [master] KUDU-2181 Raft ChangeConfig request to remove a master .. Patch Set 1: (16 comments)

[kudu-CR] KUDU-3205: Fix building scan tokens when tablet not found errors occur

2021-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16941 ) Change subject: KUDU-3205: Fix building scan tokens when tablet not found errors occur .. Patch Set 1: Code-Review+1 (5 comments)

[kudu-CR] [util] add a few new metrics in MaintenanceManager

2021-01-11 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16938 to look at the new patch set (#2). Change subject: [util] add a few new metrics in MaintenanceManager

[kudu-CR] KUDU-3205: Fix building scan tokens when tablet not found errors occur

2021-01-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16941 ) Change subject: KUDU-3205: Fix building scan tokens when tablet not found errors occur .. Patch Set 1: Code-Review+1 (1 comment)

[kudu-CR] [build] Fix the java compatibility checker script

2021-01-11 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13004 ) Change subject: [build] Fix the java compatibility checker script .. [build] Fix the java compatibility checker script This

[kudu-CR] KUDU-3205: Fix building scan tokens when tablet not found errors occur

2021-01-11 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16941 Change subject: KUDU-3205: Fix building scan tokens when tablet not found errors occur .. KUDU-3205: Fix building scan tokens when

[kudu-CR] [build] Fix the java compatibility checker script

2021-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13004 ) Change subject: [build] Fix the java compatibility checker script .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] [java] Add JAAS example to client API doc

2021-01-11 Thread Attila Bukor (Code Review)
Hello Alexey Serbin, Andrew Wong, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/16940 to review the following change. Change subject: [java] Add JAAS example to client API doc

[kudu-CR] [build] Fix the java compatibility checker script

2021-01-11 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/13004 ) Change subject: [build] Fix the java compatibility checker script .. Patch Set 4: Code-Review+1 (2 comments)

[kudu-CR] [build] Fix the java compatibility checker script

2021-01-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13004 ) Change subject: [build] Fix the java compatibility checker script .. Patch Set 3: (5 comments)

[kudu-CR] [build] Fix the java compatibility checker script

2021-01-11 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, Greg Solovyev, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13004 to look at the new patch set (#4). Change subject: [build] Fix the java compatibility checker script

[kudu-CR] [docker] Update the supported base images

2021-01-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16877 ) Change subject: [docker] Update the supported base images .. Patch Set 3: (1 comment)

[kudu-CR] [util] logging on slow execution in FindBestOp()

2021-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16938 ) Change subject: [util] logging on slow execution in FindBestOp() .. Patch Set 1: (1 comment)

[kudu-CR] [docker] Update the supported base images

2021-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16877 ) Change subject: [docker] Update the supported base images .. Patch Set 3: (1 comment)

[kudu-CR] [util] fix another lock contention in MaintenanceManager

2021-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16934 ) Change subject: [util] fix another lock contention in MaintenanceManager .. [util] fix another lock contention in

[kudu-CR] [docker] Update the supported base images

2021-01-11 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16877 ) Change subject: [docker] Update the supported base images .. [docker] Update the supported base images This patch updates the

[kudu-CR] [docker] Update the supported base images

2021-01-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16877 ) Change subject: [docker] Update the supported base images .. Patch Set 3: (1 comment)

[kudu-CR] [java] allow disabling TLS transportation from java client

2021-01-11 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16887 ) Change subject: [java] allow disabling TLS transportation from java client .. Patch Set 5: > Patch Set 5: Code-Review-2 (changed my

[kudu-CR] [java] allow disabling TLS transportation from java client

2021-01-11 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16887 ) Change subject: [java] allow disabling TLS transportation from java client .. Patch Set 5: Code-Review-2 -- To view, visit

[kudu-CR] [java] allow disabling TLS transportation from java client

2021-01-11 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16887 ) Change subject: [java] allow disabling TLS transportation from java client .. Patch Set 5: Code-Review-1 This is a nice feature, but

[kudu-CR] KUDU-1644 use range partition info for pruning

2021-01-11 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16903 ) Change subject: KUDU-1644 use range partition info for pruning .. KUDU-1644 use range partition info for pruning When pruning

[kudu-CR] [util] logging on slow execution in FindBestOp()

2021-01-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16938 ) Change subject: [util] logging on slow execution in FindBestOp() .. Patch Set 1: Code-Review+1 (1 comment)

[kudu-CR] [util] fix another lock contention in MaintenanceManager

2021-01-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16934 ) Change subject: [util] fix another lock contention in MaintenanceManager .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] [util] extra test for MaintenanceManager

2021-01-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16937 ) Change subject: [util] extra test for MaintenanceManager .. Patch Set 1: (2 comments)