StefanRRichter commented on issue #7515: [FLINK-11313][checkpoint] Introduce LZ4 compression for keyed state in full checkpoints and savepoints URL: https://github.com/apache/flink/pull/7515#issuecomment-464790827 I see, and I think it is ok that they don't have a common superclass, because if snapshot compression works or makes sense can really depend on the backend, e.g. for RocksDB incremental snapshots it would make no sense because they are already in RocksDB's native compressed format. If we are worried about the method signature, we can also introduce an interface that backends which support snapshot compression can implement. Otherwise, I think we agree about going for the configurable and extendable variant?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
