adelapena commented on code in PR #1891:
URL: https://github.com/apache/cassandra/pull/1891#discussion_r1210238363
##########
src/java/org/apache/cassandra/config/CassandraRelevantProperties.java:
##########
@@ -360,6 +361,15 @@
FLUSH_LOCAL_SCHEMA_CHANGES("cassandra.test.flush_local_schema_changes",
"true"),
TOMBSTONE_HISTOGRAM_TTL_ROUND_SECONDS("cassandra.streaminghistogram.roundseconds",
"60"),
+
+
+ /**
+ * {@link }}StorageCompatibility mode sets how the node will behave,
sstable or messaging versions to use etc according to a yaml setting.
+ * But many tests don't load the config hence we need to force it
otherwise they would run always under the default.
+ *
+ * This is a dev/CI only property. Do not use otherwise.
Review Comment:
I meant exactly that one, that the default compatibility mode is null for
some junits that don't load the config, so the compatibility mode is got from
the env var that CI/build.xml sets. What other thing did you refer to with the
"Done" comment?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
