Vishwanatha-HD commented on PR #48180: URL: https://github.com/apache/arrow/pull/48180#issuecomment-4842988863
> Thanks for testing [d97c682](https://github.com/apache/arrow/commit/d97c6820fccc895ae7b093f28e9706bda4182722) on s390x and sharing the failure log. > > This looks unrelated to the PR changes: the reported failing path is reference encode + scalar_dynamic decode for std::array<uint8_t, 3000>, which is just a byte-copy path and does not call the bit utility helpers changed here. > > To confirm, could you please run the same test on the PR base commit [a81e6c6](https://github.com/apache/arrow/commit/a81e6c6015db4ab024bfd3a2d8666bde014ba6f1)? > > ``` > ./debug/arrow-utility-test --gtest_filter=TestByteStreamSplitLargeWidth.Roundtrip > ``` > > If it also fails on the base commit, we can treat it as a separate big-endian byte-stream-split issue and merge this PR. Hi @zanmato1984.. I ran the "./debug/arrow-utility-test --gtest_filter=TestByteStreamSplitLargeWidth.Roundtrip" test on the on the PR base commit [a81e6c6].. The test is passing there.. git branch * (HEAD detached at a81e6c6015) main (venv) /golang/arrowGitRepo/arrow/cpp/build# ./debug/arrow-utility-test --gtest_filter=TestByteStreamSplitLargeWidth.Roundtrip Note: Google Test filter = TestByteStreamSplitLargeWidth.Roundtrip [==========] Running 1 test from 1 test suite. [----------] Global test environment set-up. [----------] 1 test from TestByteStreamSplitLargeWidth [ RUN ] TestByteStreamSplitLargeWidth.Roundtrip [ OK ] TestByteStreamSplitLargeWidth.Roundtrip (25 ms) [----------] 1 test from TestByteStreamSplitLargeWidth (25 ms total) [----------] Global test environment tear-down [==========] 1 test from 1 test suite ran. (25 ms total) [ PASSED ] 1 test. I also ran the test on "main" branch, without any of my patches.. This test passes there as well.. git branch * main (venv) golang/arrowGitRepo/arrow/cpp/build# ./debug/arrow-utility-test --gtest_filter=TestByteStreamSplitLargeWidth.Roundtrip Note: Google Test filter = TestByteStreamSplitLargeWidth.Roundtrip [==========] Running 1 test from 1 test suite. [----------] Global test environment set-up. [----------] 1 test from TestByteStreamSplitLargeWidth [ RUN ] TestByteStreamSplitLargeWidth.Roundtrip [ OK ] TestByteStreamSplitLargeWidth.Roundtrip (18 ms) [----------] 1 test from TestByteStreamSplitLargeWidth (18 ms total) [----------] Global test environment tear-down [==========] 1 test from 1 test suite ran. (18 ms total) [ PASSED ] 1 test. So, looks like the code changes as part of this PR has introduced the bug that we are seeing. I am invesgating it further on that.. Thanks.. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
