smiklosovic commented on code in PR #4513:
URL: https://github.com/apache/cassandra/pull/4513#discussion_r2621072986
##########
src/java/org/apache/cassandra/io/compress/CompressedSequentialWriter.java:
##########
@@ -109,7 +109,7 @@ public CompressedSequentialWriter(File file,
.bufferType(parameters.getSstableCompressor().preferredBufferType())
.finishOnClose(option.finishOnClose())
.build());
- ICompressor compressor = parameters.getSstableCompressor();
Review Comment:
alignement, plus we should most probably not use it like this. We should
still use `getSstableCompressor()` as it was before. In there, we decide what
compressor we want to use. If plugin compressor is not available, then
`decoratedSstableCompressor` will be null. The we will do
public ICompressor getSstableCompressor()
{
return pluginCompressor != null ? pluginCompressor :
sstableCompressor;
}
--
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]
