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

Sam Tunnicliffe commented on CASSANDRA-19012:
---------------------------------------------

I've been thinking about SCM and how it relates to tests. As the default is 
{{StorageCompatibilityMode.CASSANDRA_4}}, I don't think that any of the 
non-upgrade dtests are reading or writing `oa` sstables. Maybe this isn't a 
great worry as that's mostly a local concern and should be covered by unit 
tests*, but it does seem like a bit strange if dtests exclusively exercise the 
system in what should ultimately be a temporary mode. 

Regarding upgrade tests, I _think_ there is one in-jvm dtest 
({{MixedModeMixedModeTTLOverflowUpgradeTest}} which goes all the way from 4.x 
to 5.0/trunk with `StorageCompatibilityMode.NONE`, which is for the specific 
expiration issue from CASSANDRA-14227.  Is this coverage sufficient? (genuine 
question).
 
* I think this requires a specific test target to be run, so the default `ant 
test` target will never involve reads/writes of the latest SSTable format.

> It should be possible to read all supported versions of sstables regardless 
> of the compatibility mode
> -----------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-19012
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-19012
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Jacek Lewandowski
>            Priority: Normal
>             Fix For: 5.0.x
>
>
> Currently when the storage compatibility mode is set to 4, it set the current 
> write version of the ssatable to the latest compatible with 4 (N). However, 
> it additionally make new sstables (O) recognized as invalid, which is wrong. 
> We should be able to read all versions of supported sstables regardless of 
> the compatibility mode. 
> We should also ensure testing the scenario when the node tries to stream the 
> new format sstable (node with compat mode = upgrading) to a node with compat 
> mode = 4 - the test should demonstrate that such streaming is refused



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

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to