[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-11-06 Thread Abhishek Chennaka (Code Review)
Abhishek Chennaka has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20628 ) Change subject: KUDU-3519: List masters in /dump-entities .. KUDU-3519: List masters in /dump-entities Tablet servers are

[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-11-06 Thread Abhishek Chennaka (Code Review)
Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/20628 ) Change subject: KUDU-3519: List masters in /dump-entities .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.

[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-11-04 Thread Wang Xixu (Code Review)
Wang Xixu has posted comments on this change. ( http://gerrit.cloudera.org:8080/20628 ) Change subject: KUDU-3519: List masters in /dump-entities .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-11-03 Thread Marton Greber (Code Review)
Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/20628 ) Change subject: KUDU-3519: List masters in /dump-entities .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-11-03 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/20628 ) Change subject: KUDU-3519: List masters in /dump-entities .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/20628/1//CO

[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-11-03 Thread Attila Bukor (Code Review)
Hello Marton Greber, Kudu Jenkins, Abhishek Chennaka, Wang Xixu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20628 to look at the new patch set (#2). Change subject: KUDU-3519: List masters in /dump-entities ..

[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-10-31 Thread Abhishek Chennaka (Code Review)
Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/20628 ) Change subject: KUDU-3519: List masters in /dump-entities .. Patch Set 1: +1 for adding a test. Apart from that and the concerns p

[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-10-26 Thread Wang Xixu (Code Review)
Wang Xixu has posted comments on this change. ( http://gerrit.cloudera.org:8080/20628 ) Change subject: KUDU-3519: List masters in /dump-entities .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/20628/1//COMMI

[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-10-26 Thread Marton Greber (Code Review)
Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/20628 ) Change subject: KUDU-3519: List masters in /dump-entities .. Patch Set 1: (1 comment) Overall looks good to me, thanks for the patch!

[kudu-CR] KUDU-3519: List masters in /dump-entities

2023-10-26 Thread Attila Bukor (Code Review)
Hello Marton Greber, Abhishek Chennaka, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/20628 to review the following change. Change subject: KUDU-3519: List masters in /dump-entities .. KUD