yifan-c commented on code in PR #4475:
URL: https://github.com/apache/cassandra/pull/4475#discussion_r2539492182


##########
src/java/org/apache/cassandra/io/compress/ZstdCompressor.java:
##########
@@ -22,11 +22,14 @@
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 
+import com.github.luben.zstd.Zstd;
+
 /**
  * ZSTD Compressor
  */
 public class ZstdCompressor extends ZstdCompressorBase implements ICompressor
 {
+    public static final int BEST_COMPRESSION_LEVEL = 
Zstd.maxCompressionLevel();

Review Comment:
   Thank you for making `ZstdCompressor` backward compatible! 
   
   Maybe add a comment to explain why there are 2 different 
`BEST_COMPRESSION_LEVEL`



-- 
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]

Reply via email to