[kudu-CR](branch-1.17.x) Fix CheckHolePunch for bigger than 4k blocks.

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: Fix CheckHolePunch for bigger than 4k blocks. .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/20930 To unsubscribe,

[kudu-CR](branch-1.17.x) Fix CheckHolePunch for bigger than 4k blocks.

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20930 ) Change subject: Fix CheckHolePunch for bigger than 4k blocks. .. Patch Set 1: Verified+1 unrelated test failure in

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

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin 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 7: (9 comments) Overall looks good to me, but I

[kudu-CR](branch-1.17.x) Fix CheckHolePunch for bigger than 4k blocks.

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20930 Change subject: Fix CheckHolePunch for bigger than 4k blocks. .. Fix CheckHolePunch for bigger than 4k blocks. With the new 64k ARM

[kudu-CR] Fix CheckHolePunch for bigger than 4k blocks.

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20894 ) Change subject: Fix CheckHolePunch for bigger than 4k blocks. .. Patch Set 5: Code-Review+2 Thank you for the fix! I verified it

[kudu-CR] Fix CheckHolePunch for bigger than 4k blocks.

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20894 ) Change subject: Fix CheckHolePunch for bigger than 4k blocks. .. Fix CheckHolePunch for bigger than 4k blocks. With the new

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

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin 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 7: (1 comment)

[kudu-CR](branch-1.17.x) KUDU-3526 [java] Scanner should bind with a tserver in java client.

2024-01-19 Thread Abhishek Chennaka (Code Review)
Abhishek Chennaka has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20923 ) Change subject: KUDU-3526 [java] Scanner should bind with a tserver in java client. .. KUDU-3526 [java] Scanner should

[kudu-CR](branch-1.17.x) KUDU-3491 Destruct master before creating a new one

2024-01-19 Thread Abhishek Chennaka (Code Review)
Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/20917 ) Change subject: KUDU-3491 Destruct master before creating a new one .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.17.x) KUDU-3526 [java] Scanner should bind with a tserver in java client.

2024-01-19 Thread Abhishek Chennaka (Code Review)
Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/20923 ) Change subject: KUDU-3526 [java] Scanner should bind with a tserver in java client. .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR](branch-1.17.x) KUDU-3491 Destruct master before creating a new one

2024-01-19 Thread Abhishek Chennaka (Code Review)
Abhishek Chennaka has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20917 ) Change subject: KUDU-3491 Destruct master before creating a new one .. KUDU-3491 Destruct master before creating a new one

[kudu-CR](branch-1.17.x) KUDU-3526 [java] Scanner should bind with a tserver in java client.

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: KUDU-3526 [java] Scanner should bind with a tserver in java client. .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR](branch-1.17.x) KUDU-3526 [java] Scanner should bind with a tserver in java client.

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20923 ) Change subject: KUDU-3526 [java] Scanner should bind with a tserver in java client. .. Patch Set 1: Verified+1 unrelated test

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

2024-01-19 Thread Zoltan Martonka (Code Review)
Hello Mahesh Reddy, Alexey Serbin, Ashwani Raina, Kudu Jenkins, Abhishek Chennaka, Wang Xixu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20725 to look at the new patch set (#7). Change subject: KUDU-3527 Fix block manager test on 64k filesystems

[kudu-CR] Fix typos

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20927 ) Change subject: Fix typos .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/20927 To unsubscribe,

[kudu-CR] Fix typos

2024-01-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20927 ) Change subject: Fix typos .. Fix typos Change-Id: I71524db97455bb27db44b5820cbe81fd95e207d9 Reviewed-on:

[kudu-CR] Fix CheckHolePunch for bigger than 4k blocks.

2024-01-19 Thread Zoltan Martonka (Code Review)
Zoltan Martonka has posted comments on this change. ( http://gerrit.cloudera.org:8080/20894 ) Change subject: Fix CheckHolePunch for bigger than 4k blocks. .. Patch Set 5: (1 comment)

[kudu-CR] Fix typos

2024-01-19 Thread Code Review
Ádám Bakai has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20927 Change subject: Fix typos .. Fix typos Change-Id: I71524db97455bb27db44b5820cbe81fd95e207d9 --- M src/kudu/fs/dir_manager.cc M