Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22515 )

Change subject: fix(test): ensure proper alignment of `version_` in `NodeBase` 
on ARM64
......................................................................


Patch Set 3:

(1 comment)

Thank you for looking at this!

Just to clarify: are you sure that the source code used to compile cbtree-test 
included changelist 
https://github.com/apache/kudu/commit/2161950da43f9daa71f2297683e60e3ff7fcd1db?

http://gerrit.cloudera.org:8080/#/c/22515/3//COMMIT_MSG
Commit Message:

PS3:
Could you share a bit more information about the version of the source files 
that you used to build the cbtree-test binary that would fail with the stack 
trace captured?

AFAIK, changelist [1] should have addressed issues like that, and there 
shouldn't be base::subtle::Acquire_Load() symbol in the stack that involves 
btree::VersionField::Lock() call.

[1] 
https://github.com/apache/kudu/commit/2161950da43f9daa71f2297683e60e3ff7fcd1db



--
To view, visit http://gerrit.cloudera.org:8080/22515
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id082d3ca243418d4f0016fbd21a59ead0fa9dfc5
Gerrit-Change-Number: 22515
Gerrit-PatchSet: 3
Gerrit-Owner: Yan-Daojiang <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 25 Feb 2025 03:48:25 +0000
Gerrit-HasComments: Yes

Reply via email to