Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20613 )
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 on x86 architecture it is 4k but on aarch64 some linux distribution (eg. 4.19.90-23.30.v2101.ky10.aarch6) can use a 64k block size kernel. Test EncryptionEnabled/LogBlockManagerTest.MetricsTest/1 run fails in some 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 Reviewed-on: http://gerrit.cloudera.org:8080/20613 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <ale...@apache.org> --- M src/kudu/fs/log_block_manager-test.cc 1 file changed, 9 insertions(+), 5 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- 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: merged Gerrit-Change-Id: Ide52a251b15b1af437d570c146beb0c30fed161b Gerrit-Change-Number: 20613 Gerrit-PatchSet: 12 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>
