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 (#10).

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
---
M src/kudu/fs/log_block_manager-test.cc
1 file changed, 13 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/20613/10
--
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: 10
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>

Reply via email to