[kudu-CR] KUDU-3486 Periodically send tombstoned replica report to delete it.

2023-11-29 Thread Yifan Zhang (Code Review)
Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/20600 ) Change subject: KUDU-3486 Periodically send tombstoned replica report to delete it. .. Patch Set 20: (2 comments) http://gerrit.cloude

[kudu-CR] KUDU-3527 Fix block manager test on 64k filesystems

2023-11-29 Thread Mahesh Reddy (Code Review)
Mahesh Reddy has posted comments on this change. ( http://gerrit.cloudera.org:8080/20725 ) Change subject: KUDU-3527 Fix block manager test on 64k filesystems .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/20

[kudu-CR] [cfile] remove redundant StringPrintf in status messages

2023-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20740 ) Change subject: [cfile] remove redundant StringPrintf in status messages .. [cfile] remove redundant StringPrintf in status mes

[kudu-CR] [cfile] remove redundant StringPrintf in status messages

2023-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20740 ) Change subject: [cfile] remove redundant StringPrintf in status messages .. Patch Set 2: > Patch Set 2: Code-Review+2 Thanks a lot fo

[kudu-CR] [cfile] remove redundant StringPrintf in status messages

2023-11-29 Thread Yingchun Lai (Code Review)
Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/20740 ) Change subject: [cfile] remove redundant StringPrintf in status messages .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit

[kudu-CR] [tools] disambiguate parse metrics' output

2023-11-29 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Abhishek Chennaka, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20741 to look at the new patch set (#2). Change subject: [tools] disambiguate parse_metrics' output ..

[kudu-CR] [cfile] remove redundant StringPrintf in status messages

2023-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20740 ) Change subject: [cfile] remove redundant StringPrintf in status messages .. Patch Set 2: Verified+1 unrelated test failure: a well-kno

[kudu-CR] [cfile] remove redundant StringPrintf in status messages

2023-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [cfile] remove redundant StringPrintf in status messages .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/20740 To uns

[kudu-CR] [cfile] remove redundant StringPrintf in status messages

2023-11-29 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Abhishek Chennaka, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20740 to look at the new patch set (#2). Change subject: [cfile] remove redundant StringPrintf in status messages .

[kudu-CR] [tools] disambiguate parse metrics' output

2023-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20741 Change subject: [tools] disambiguate parse_metrics' output .. [tools] disambiguate parse_metrics' output This patch improves the output

[kudu-CR] [cfile] remove redundant StringPrintf in status messages

2023-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20740 Change subject: [cfile] remove redundant StringPrintf in status messages .. [cfile] remove redundant StringPrintf in status messages Ch

[kudu-CR] [arm64] Fix CountOnes() function

2023-11-29 Thread Ashwani Raina (Code Review)
Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/20736 ) Change subject: [arm64] Fix CountOnes() function .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/20736/2//COMMIT_MSG

[kudu-CR] KUDU-3524 Fix core of sending keep-alive requests periodically

2023-11-29 Thread Ashwani Raina (Code Review)
Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/20739 ) Change subject: KUDU-3524 Fix core of sending keep-alive requests periodically .. Patch Set 2: (4 comments) http://gerrit.cloudera.or

[kudu-CR] [messenger] Initialize SASL proto name to principal

2023-11-29 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/20708 to look at the new patch set (#6). Change subject: [messenger] Initialize SASL proto name to principal

[kudu-CR] [messenger] Initialize SASL proto name to principal

2023-11-29 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/20708 to look at the new patch set (#4). Change subject: [messenger] Initialize SASL proto name to principal

[kudu-CR] [messenger] Initialize SASL proto name to principal

2023-11-29 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/20708 to look at the new patch set (#5). Change subject: [messenger] Initialize SASL proto name to principal

[kudu-CR] [ExternalMiniCluster] improve leader master()

2023-11-29 Thread Code Review
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20714 to look at the new patch set (#9). Change subject: [ExternalMiniCluster] improve leader_master() ..

[kudu-CR] KUDU-3526 [java] Scanner should bind with a tserver in java client.

2023-11-29 Thread Song Jiacheng (Code Review)
Hello Marton Greber, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20715 to look at the new patch set (#11). Change subject: KUDU-3526 [java] Scanner should bind with a tserver in java client.

[kudu-CR] KUDU-3498 Scanner keeps alive in periodically

2023-11-29 Thread Wang Xixu (Code Review)
Wang Xixu has posted comments on this change. ( http://gerrit.cloudera.org:8080/20282 ) Change subject: KUDU-3498 Scanner keeps alive in periodically .. Patch Set 25: > Patch Set 25: > > > > Patch Set 25: > > > > > > > (3 co

[kudu-CR] KUDU-3526 [java] Scanner should bind with a tserver in java client.

2023-11-29 Thread Song Jiacheng (Code Review)
Hello Marton Greber, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20715 to look at the new patch set (#10). Change subject: KUDU-3526 [java] Scanner should bind with a tserver in java client.

[kudu-CR] KUDU-3524 Fix core of sending keep-alive requests periodically

2023-11-29 Thread Wang Xixu (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20739 to look at the new patch set (#2). Change subject: KUDU-3524 Fix core of sending keep-alive requests periodically

[kudu-CR] KUDU-3526 [java] Scanner should bind with a tserver in java client.

2023-11-29 Thread Song Jiacheng (Code Review)
Hello Marton Greber, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20715 to look at the new patch set (#9). Change subject: KUDU-3526 [java] Scanner should bind with a tserver in java client. .

[kudu-CR] KUDU-3524 Fix core of sending keep-alive requests periodically

2023-11-29 Thread Wang Xixu (Code Review)
Wang Xixu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20739 Change subject: KUDU-3524 Fix core of sending keep-alive requests periodically .. KUDU-3524 Fix core of sending keep-alive requests periodic