[ 
https://issues.apache.org/jira/browse/CASSANDRA-17723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17741938#comment-17741938
 ] 

Michael Semb Wever commented on CASSANDRA-17723:
------------------------------------------------

Thanks for your patience [~shahidhs] and [~Nayana].

I have updated your patch for trunk, and made a few trivial changes for code 
style, here: 
https://github.com/apache/cassandra/compare/trunk...thelastpickle:cassandra:mck/17723/trunk

Can you please test and let me know if this works for you.

I will need a second reviewer are well for this, it being a low-level change.

> Add support for big endian architecture (s390x)
> -----------------------------------------------
>
>                 Key: CASSANDRA-17723
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-17723
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Legacy/Observability, Legacy/Testing
>            Reporter: Shahid Shaikh
>            Assignee: Shahid Shaikh
>            Priority: Normal
>             Fix For: 5.x
>
>         Attachments: CASSANDRA-17723-4.0.patch
>
>
> Noticed that some of the Cassandra tests are failing on big endian 
> architectures (s390x). Please find the attached code patch which corrects the 
> data handling for big endian architecture. It also corrects the byte ordering 
> when dealing with sun.misc.Unsafe methods. After the patch following tests 
> are passing for s390x which were failing earlier:
> TTLTest
> ScrubTest
> LegacySSTableTest
> SSTableExportSchemaLoadingTest
> SSTableMetadataViewerTest
> StandaloneUpgraderOnSStablesTest
> StandaloneVerifierOnSSTablesTest
> The code change ensures that Cassandra for little endian architectures remain 
> unaffected.
>  



--
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