Hello Alexey Serbin, Zoltan Martonka, Yingchun Lai, Attila Bukor, Yifan Zhang, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/20613 to look at the new patch set (#7). Change subject: Fix a unit test in aarch64 system ...................................................................... Fix a unit test in aarch64 system The block size is not fixed in different architecture systems. For example, usually in x86 architecture it is 4k but in aarch64 64k by default, for example, in system 4.19.90-23.30.v2101.ky10.aarch6 is 64k. Test EncryptionEnabled/LogBlockManagerTest.MetricsTest/1 run fails in aarch64 systems currently because of ignoring the block size of the architecture. Instead of setting a fixed value for FLAGS_log_container_max_size, it is better to use the variable block size to set it. Change-Id: Ide52a251b15b1af437d570c146beb0c30fed161b --- M src/kudu/fs/log_block_manager-test.cc 1 file changed, 6 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/20613/7 -- To view, visit http://gerrit.cloudera.org:8080/20613 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ide52a251b15b1af437d570c146beb0c30fed161b Gerrit-Change-Number: 20613 Gerrit-PatchSet: 7 Gerrit-Owner: Wang Xixu <1450306...@qq.com> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Wang Xixu <1450306...@qq.com> Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com> Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org> Gerrit-Reviewer: Zoltan Martonka <zmarto...@cloudera.com>