Alexey Serbin has submitted this change and it was merged. ( 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 core, you can use filesystems with up to 64k block size. Currently Kudu does not start if the data is on a filesystem with larger than 4k blocks. Problem: CheckHolePunch has hard-coded 4k block size instead of using the get env->GetBlockSize. Kudu seems to be running fine on a 64k fs after this fix. More info: https://kudu.apache.org/docs/troubleshooting.html#req_hole_punching Change-Id: Ib4af73f0aa25db674fe0a34355cecd27a0c68417 Reviewed-on: http://gerrit.cloudera.org:8080/20894 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <ale...@apache.org> (cherry picked from commit de87aca3dd29177658790cffeb7a1de0c8d69231) Reviewed-on: http://gerrit.cloudera.org:8080/20930 Tested-by: Alexey Serbin <ale...@apache.org> Reviewed-by: Wang Xixu <1450306...@qq.com> Reviewed-by: Yingchun Lai <laiyingc...@apache.org> --- M src/kudu/fs/dir_util.cc 1 file changed, 8 insertions(+), 6 deletions(-) Approvals: Alexey Serbin: Verified Wang Xixu: Looks good to me, but someone else must approve Yingchun Lai: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20930 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: Ib4af73f0aa25db674fe0a34355cecd27a0c68417 Gerrit-Change-Number: 20930 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Wang Xixu <1450306...@qq.com> Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org> Gerrit-Reviewer: Zoltan Martonka <zmarto...@cloudera.com>