Yan-Daojiang 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)

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 file
I'm using the 1.17 branch and now trying to cherry-pick the changelist[1]. 
Although the base::subtle::Acquire_Load() symbol will not appear on the stack, 
the use case still fails.
[2] shows the error stack.

BTW, aren't unit tests for the ARM platform included in Community CI? It seems 
that there are several unit test cases that cannot pass or are unstable on the 
ARM platform. Can the community help clarify which use cases they are? I will 
try to fix them.

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

[2]
Note: Google Test filter = TestCBTree.TestLeafNode
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from TestCBTree
[ RUN      ] TestCBTree.TestLeafNode
*** Aborted at 1740481901 (unix time) try "date -d @1740481901" if you are 
using GNU date ***
PC: @                0x0 (unknown)
*** SIGBUS (@0xffffe86ac31f) received by PID 25531 (TID 0xffffa1d60040) from 
PID 18446744073313895199; stack trace: ***
    @     0xffffa23ee890 google::(anonymous namespace)::FailureSignalHandler()
    @     0xffffa38407a0 ([vdso]+0x79f)
    @           0x26a544 kudu::tablet::btree::VersionField::Lock()
    @           0x2712f4 kudu::tablet::btree::NodeBase<>::Lock()
    @           0x268e9c kudu::tablet::btree::TestCBTree::InsertInLeaf<>()
    @           0x25c2a8 
kudu::tablet::btree::TestCBTree_TestLeafNode_Test::TestBody()
    @     0xffffa22b05f8 
testing::internal::HandleExceptionsInMethodIfSupported<>()
    @     0xffffa22a2328 testing::Test::Run()
    @     0xffffa22a24c8 testing::TestInfo::Run()
    @     0xffffa22a2b70 testing::TestSuite::Run()
    @     0xffffa22a7264 testing::internal::UnitTestImpl::RunAllTests()
    @     0xffffa22a25d0 testing::UnitTest::Run()
    @     0xffffa37c5fa4 (unknown)
    @     0xffffa37c53d4 (unknown)
    @     0xffffa1dac79c __libc_start_call_main
    @     0xffffa1dac86c __libc_start_main_alias_2
    @           0x25be30 _start



--
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-Reviewer: Yan-Daojiang <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
Gerrit-Comment-Date: Wed, 26 Feb 2025 02:50:57 +0000
Gerrit-HasComments: Yes

Reply via email to