aratno commented on code in PR #4118:
URL: https://github.com/apache/cassandra/pull/4118#discussion_r2067133681
##########
src/java/org/apache/cassandra/io/sstable/AbstractSSTableSimpleWriter.java:
##########
@@ -112,14 +117,15 @@ protected SSTableTxnWriter createWriter(SSTable.Owner
owner) throws IOException
SerializationHeader header = new SerializationHeader(true,
metadata.get(), columns, EncodingStats.NO_STATS);
if (makeRangeAware)
- return SSTableTxnWriter.createRangeAware(metadata, 0,
ActiveRepairService.UNREPAIRED_SSTABLE, ActiveRepairService.NO_PENDING_REPAIR,
false, format, header);
+ return SSTableTxnWriter.createRangeAware(metadata, 0,
ActiveRepairService.UNREPAIRED_SSTABLE, ActiveRepairService.NO_PENDING_REPAIR,
false, MutationIdRanges.NONE, format, header);
Review Comment:
This is used for unit tests that produce SSTables, and by the
cassandra-analytics write path. I've updated to fail early with a comment
indicating upcoming support via coordinated-import.
--
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]
