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

Jon Haddad commented on CASSANDRA-21462:
----------------------------------------

Closing this as complete.  Merged as 5b340684e6f30536a850a646afe8076b1564195e 
and 3158c2d2ff6846766086fae51ac2924fbc41d7e9

> Add byte for byte test harness to detect and resolve cursor deviations from 
> iterator path
> -----------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-21462
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-21462
>             Project: Apache Cassandra
>          Issue Type: Sub-task
>          Components: Local/Compaction
>            Reporter: Jon Haddad
>            Assignee: Jon Haddad
>            Priority: High
>             Fix For: 6.0
>
>         Attachments: 
> ci_summary_rustyrazorblade_21462-cursor-compaction-6.0-stabilization_123.html,
>  ci_summary_rustyrazorblade_jhaddad-CASSANDRA-21462-trunk_680.html
>
>
> This issue addresses various deviations and inconsistencies with the cursor 
> path by introducing tests that verify both the old iterator path and the new 
> cursor path generate the same outputs given the same inputs.  It also fixes 
> various issues found in the process of creating this test system that 
> resulted in corruption or incorrect values being carried into the resulting 
> SSTable.
> It adds a test that uses the JVM's internal memory tracking to assert that we 
> do NOT do unnecessary allocations in the cursor path that should guard 
> against future regressions, and fixes an unnecessary enum allocation.
> It does NOT add support for BTI, multi-cell columns, counters, etc.
>  



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