João Reis created CASSGO-128:
--------------------------------

             Summary: LZ4 test fails on arm64
                 Key: CASSGO-128
                 URL: https://issues.apache.org/jira/browse/CASSGO-128
             Project: Apache Cassandra Go driver
          Issue Type: Bug
          Components: Core
            Reporter: João Reis


[~tolbertam] posted this:
{quote}Doing some sanity checking of the GoCQL 2.1.2 test and ran into some 
weirdness with {{lz4_test:TestLz4Compressor}}



=== RUN TestLZ4Compressor lz4_test.go:46: Error Trace: 
/dist.apache.org/repos/dist/dev/cassandra/cassandra-gocql-driver/2.1.2/apache-cassandra-gocql-driver-2.1.2/lz4/lz4_test.go:46
 Error: An error is expected but got nil. Test: TestLZ4Compressor


it looks related to this: 
[https://github.com/apache/cassandra-gocql-driver/pull/1662] / 
[https://github.com/pierrec/lz4/commit/f27c1a0ed0f6c78a37c7b904fe041be5b5d6a93e]
 , although i'm not sure why it fails when we are running an older version of 
pierrec/lz4 that doesn't have this change.   This fails on 2.1.1 as well, so 
not related to this release.

It fails for me both in a docker container and on my host Mac OS system.   I 
realized that maybe its some kind of weirdness with arm64, and tried it in a 
docker container with platform linux/amd64 and everything runs fine:



=== RUN TestLZ4Compressor --- PASS: TestLZ4Compressor (0.00s)


Wondering if we could consider picking up the change in #1662 in a future 
release, anyways as it might be nice to pick up a newer version anyways since 
lz4 v4.1.8 is a bit old anyways.{quote}
 
It looks like upgrading the lz4 dependency is enough to fix this. Maybe we 
should add some arm64 testing to the matrix as well.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to