Zihao Ye has uploaded this change for review. ( http://gerrit.cloudera.org:8080/22147
Change subject: IMPALA-13590: Use CacheLineAligned instead of CACHELINE_ALIGNED for PerFilterState ...................................................................... IMPALA-13590: Use CacheLineAligned instead of CACHELINE_ALIGNED for PerFilterState Currently, class PerFilterState is marked with CACHELINE_ALIGNED, which is actually a macro definition of __attribute__((aligned())). For static allocation, it can specify memory alignment, but it cannot specify alignment for dynamic memory allocation. In the code, all PerFilterState objects are dynamically created using make_unique in the function RuntimeFilterBank::BuildFilterMap(), so CACHELINE_ALIGNED does not really have any effect. It might even lead to unexpected situations as described in the JIRA comments. Therefore, this patch replaces CACHELINE_ALIGNED with CacheLineAligned, which supports alignment for dynamic memory allocation. Testing: - Existing tests cover this change. Change-Id: I0d119679b1d446cb69f6eec595def2d40cb6d1ff --- M be/src/runtime/runtime-filter-bank.h 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/47/22147/1 -- To view, visit http://gerrit.cloudera.org:8080/22147 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I0d119679b1d446cb69f6eec595def2d40cb6d1ff Gerrit-Change-Number: 22147 Gerrit-PatchSet: 1 Gerrit-Owner: Zihao Ye <[email protected]>
