[kudu-CR] jwt: plumb JWT into mini cluster

2023-03-14 Thread Zoltan Chovan (Code Review)
Zoltan Chovan has uploaded a new patch set (#7) to the change originally created by Andrew Wong. ( http://gerrit.cloudera.org:8080/18475 ) Change subject: jwt: plumb JWT into mini cluster .. jwt: plumb JWT into mini cluster Thi

[kudu-CR] jwt: introduce MiniOidc

2023-03-14 Thread Zoltan Chovan (Code Review)
Zoltan Chovan has uploaded a new patch set (#13) to the change originally created by Andrew Wong. ( http://gerrit.cloudera.org:8080/18473 ) Change subject: jwt: introduce MiniOidc .. jwt: introduce MiniOidc This patch takes som

[kudu-CR] [java] KUDU-3455 Reduce space complexity about pruning hash partitions for in-list predicate

2023-03-14 Thread Yifan Zhang (Code Review)
Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/19568 ) Change subject: [java] KUDU-3455 Reduce space complexity about pruning hash partitions for in-list predicate .. Patch Set 6: (4 comment

[kudu-CR] jwt: introduce MiniOidc

2023-03-14 Thread Yuqi Du (Code Review)
Yuqi Du has posted comments on this change. ( http://gerrit.cloudera.org:8080/18473 ) Change subject: jwt: introduce MiniOidc .. Patch Set 12: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/18473 To unsubscri

[kudu-CR] jwt: introduce MiniOidc

2023-03-14 Thread Wenzhe Zhou (Code Review)
Wenzhe Zhou has posted comments on this change. ( http://gerrit.cloudera.org:8080/18473 ) Change subject: jwt: introduce MiniOidc .. Patch Set 12: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/18473 To unsub

[kudu-CR] jwt: introduce MiniOidc

2023-03-14 Thread Zoltan Chovan (Code Review)
Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/18473 ) Change subject: jwt: introduce MiniOidc .. Patch Set 12: (2 comments) http://gerrit.cloudera.org:8080/#/c/18473/11/src/kudu/util/jwt-

[kudu-CR] jwt: introduce MiniOidc

2023-03-14 Thread Zoltan Chovan (Code Review)
Zoltan Chovan has uploaded a new patch set (#12) to the change originally created by Andrew Wong. ( http://gerrit.cloudera.org:8080/18473 ) Change subject: jwt: introduce MiniOidc .. jwt: introduce MiniOidc This patch takes som

[kudu-CR] jwt: Additional test

2023-03-14 Thread Zoltan Chovan (Code Review)
Hello Marton Greber, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Wenzhe Zhou, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19156 to look at the new patch set (#2). Change subject: jwt: Additional test .

[kudu-CR] jwt: plumb JWT into mini cluster

2023-03-14 Thread Zoltan Chovan (Code Review)
Zoltan Chovan has uploaded a new patch set (#6) to the change originally created by Andrew Wong. ( http://gerrit.cloudera.org:8080/18475 ) Change subject: jwt: plumb JWT into mini cluster .. jwt: plumb JWT into mini cluster Thi

[kudu-CR] [Tool] Show the information of a tablet

2023-03-14 Thread Wang Xixu (Code Review)
Wang Xixu has posted comments on this change. ( http://gerrit.cloudera.org:8080/19501 ) Change subject: [Tool] Show the information of a tablet .. Patch Set 7: (11 comments) http://gerrit.cloudera.org:8080/#/c/19501/5//COMMIT

[kudu-CR] [Tool] Show the information of a tablet

2023-03-14 Thread Wang Xixu (Code Review)
Hello Tidy Bot, Alexey Serbin, Yuqi Du, Yingchun Lai, Kudu Jenkins, KeDeng, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19501 to look at the new patch set (#7). Change subject: [Tool] Show the information of a tablet .

[kudu-CR] [server] add 'slow scans' metric for tablet server

2023-03-14 Thread KeDeng (Code Review)
Hello Alexey Serbin, Ashwani Raina, Yingchun Lai, Kudu Jenkins, Wang Xixu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19545 to look at the new patch set (#11). Change subject: [server] add 'slow_scans' metric for tablet server .

[kudu-CR] [server] add 'slow scans' metric for tablet server

2023-03-14 Thread KeDeng (Code Review)
KeDeng has posted comments on this change. ( http://gerrit.cloudera.org:8080/19545 ) Change subject: [server] add 'slow_scans' metric for tablet server .. Patch Set 10: (8 comments) Thanks for your reviews. http://gerrit.clo

[kudu-CR] [master] exclude tservers at MAINTENANCE MODE when leader rebalancing

2023-03-14 Thread KeDeng (Code Review)
KeDeng has posted comments on this change. ( http://gerrit.cloudera.org:8080/19608 ) Change subject: [master] exclude tservers at MAINTENANCE_MODE when leader rebalancing .. Patch Set 5: (3 comments) http://gerrit.cloudera.o

[kudu-CR] [ut] make a checking command of 'TestTServerListState' more meaningful

2023-03-14 Thread KeDeng (Code Review)
KeDeng has posted comments on this change. ( http://gerrit.cloudera.org:8080/19611 ) Change subject: [ut] make a checking command of 'TestTServerListState' more meaningful .. Patch Set 4: (1 comment) http://gerrit.cloudera.o

[kudu-CR] [Tool] Show the information of a tablet

2023-03-14 Thread Wang Xixu (Code Review)
Hello Alexey Serbin, Yuqi Du, Yingchun Lai, Kudu Jenkins, KeDeng, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19501 to look at the new patch set (#6). Change subject: [Tool] Show the information of a tablet ...

[kudu-CR] Exit external mini cluster services when test is stopped

2023-03-14 Thread Code Review
Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19613 to look at the new patch set (#2). Change subject: Exit external_mini_cluster services when test is stopped ...

[kudu-CR] KUDU-3448 Add support for encrypting IPKI keys

2023-03-14 Thread Attila Bukor (Code Review)
Hello Mahesh Reddy, Tidy Bot, Zoltan Chovan, Alexey Serbin, Ashwani Raina, Kudu Jenkins, Abhishek Chennaka, Ádám Bakai, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19616 to look at the new patch set (#4). Change subject: KUDU-3448 Add support for encryp

[kudu-CR] KUDU-3448 Add support for encrypting IPKI keys

2023-03-14 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/19616 ) Change subject: KUDU-3448 Add support for encrypting IPKI keys .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/19616/3

[kudu-CR] KUDU-3448 Plumbing for encrypting key material

2023-03-14 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/19615 ) Change subject: KUDU-3448 Plumbing for encrypting key material .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/19615/3

[kudu-CR] KUDU-3448 Plumbing for encrypting key material

2023-03-14 Thread Attila Bukor (Code Review)
Hello Mahesh Reddy, Tidy Bot, Zoltan Chovan, Alexey Serbin, Ashwani Raina, Kudu Jenkins, Abhishek Chennaka, Ádám Bakai, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19615 to look at the new patch set (#4). Change subject: KUDU-3448 Plumbing for encryptin

[kudu-CR] [ut] make a checking command of 'TestTServerListState' more meaningful

2023-03-14 Thread Yingchun Lai (Code Review)
Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/19611 ) Change subject: [ut] make a checking command of 'TestTServerListState' more meaningful .. Patch Set 4: Code-Review+1 (1 comment) http

[kudu-CR] [ut] make a checking command of 'TestTServerListState' more meaningful

2023-03-14 Thread Yuqi Du (Code Review)
Yuqi Du has posted comments on this change. ( http://gerrit.cloudera.org:8080/19611 ) Change subject: [ut] make a checking command of 'TestTServerListState' more meaningful .. Patch Set 4: (1 comment) > Patch Set 3: > > (1 c

[kudu-CR] [ut] make a checking command of 'TestTServerListState' more meaningful

2023-03-14 Thread Yuqi Du (Code Review)
Hello Yingchun Lai, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19611 to look at the new patch set (#4). Change subject: [ut] make a checking command of 'TestTServerListState' more meaningful ...

[kudu-CR] [java] KUDU-3455 Reduce space complexity about pruning hash partitions for in-list predicate

2023-03-14 Thread Yuqi Du (Code Review)
Yuqi Du has posted comments on this change. ( http://gerrit.cloudera.org:8080/19568 ) Change subject: [java] KUDU-3455 Reduce space complexity about pruning hash partitions for in-list predicate .. Patch Set 6: (3 comments)

[kudu-CR] [java] KUDU-3455 Reduce space complexity about pruning hash partitions for in-list predicate

2023-03-14 Thread Yuqi Du (Code Review)
Hello Mahesh Reddy, Alexey Serbin, Ashwani Raina, Yingchun Lai, Yifan Zhang, Kudu Jenkins, Wang Xixu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19568 to look at the new patch set (#6). Change subject: [java] KUDU-3455 Reduce space complexity about pru

[kudu-CR] [Tool] Compact metadata file for old versions of Kudu

2023-03-14 Thread Wang Xixu (Code Review)
Hello Tidy Bot, Yingchun Lai, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19327 to look at the new patch set (#5). Change subject: [Tool] Compact metadata file for old versions of Kudu ...

[kudu-CR] KUDU-3448 Plumbing for encrypting key material

2023-03-14 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/19615 ) Change subject: KUDU-3448 Plumbing for encrypting key material .. Patch Set 3: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8

[kudu-CR] [Tool] Show the information of a tablet

2023-03-14 Thread Yingchun Lai (Code Review)
Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/19501 ) Change subject: [Tool] Show the information of a tablet .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/19501/5//COMMI

[kudu-CR] [server] add 'slow scans' metric for tablet server

2023-03-14 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/19545 ) Change subject: [server] add 'slow_scans' metric for tablet server .. Patch Set 10: (8 comments) http://gerrit.cloudera.org:8080/#/c/