adelapena commented on code in PR #2391:
URL: https://github.com/apache/cassandra/pull/2391#discussion_r1221489709


##########
test/unit/org/apache/cassandra/config/DatabaseDescriptorTest.java:
##########
@@ -803,6 +805,15 @@ public void testDefaultSslContextFactoryConfiguration()
         
Assert.assertTrue(config.server_encryption_options.ssl_context_factory.parameters.isEmpty());
     }
 
+    @Test
+    public void testBtiFormatAndStorageCompatibilityMode()

Review Comment:
   Actually, we can test all compatibility modes against bti and big, and make 
sure that the test will be updated on new compatibility modes: 
https://github.com/adelapena/cassandra/blob/30565119702200d7dac215abef33ebc123d8e62c/test/unit/org/apache/cassandra/config/DatabaseDescriptorTest.java#L810-L835



-- 
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: pr-unsubscr...@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to