Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20678 )
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 <[email protected]> (cherry picked from commit 976e9fc1105c2769d56a2ed32137d4969c8374bd) Reviewed-on: http://gerrit.cloudera.org:8080/20678 Reviewed-by: Yingchun Lai <[email protected]> Reviewed-by: Wang Xixu <[email protected]> --- M src/kudu/fs/log_block_manager-test.cc 1 file changed, 9 insertions(+), 5 deletions(-) Approvals: Kudu Jenkins: Verified Yingchun Lai: Looks good to me, approved Wang Xixu: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/20678 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: Ide52a251b15b1af437d570c146beb0c30fed161b Gerrit-Change-Number: 20678 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
