jacek-lewandowski commented on code in PR #2267:
URL: https://github.com/apache/cassandra/pull/2267#discussion_r1163877511
##########
conf/cassandra.yaml:
##########
@@ -1924,8 +1924,14 @@ drop_compact_storage_enabled: false
# which is used in some serialization to denote the format type in a compact
way (such as local key cache); and 'name'
# which will be used to recognize the format type - in particular that name
will be used in sstable file names and in
# stream headers so the name has to be the same for the same format across all
the nodes in the cluster.
+# The first entry in this list is the format that will be used for
newly-created SSTables. The other formats given
+# will be used to read any SSTables present in the data directories or
streamed.
sstable_formats:
- class_name: org.apache.cassandra.io.sstable.format.big.BigFormat
parameters:
id: 0
name: big
+ - class_name: org.apache.cassandra.io.sstable.format.bti.BtiFormat
+ parameters:
+ id: 1
+ name: bti
Review Comment:
I'll fix the configuration in CASSANDRA-18441, I agree it is bad and was
aimed too much to make everything configurable.
(nit: @dcapwell that map is `ImmutableMap` which retains order, OTOH that's
too bad that `DatabaseDescriptor` does not make it explicit and returns just a
`Map`)
--
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]
