[ 
https://issues.apache.org/jira/browse/FLINK-14159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16936548#comment-16936548
 ] 

jackylau commented on FLINK-14159:
----------------------------------

[~klion26] Does that mean checkpoint has twice compression when we use 
SnappyStreamCompressionDecorator. The first is checkpoint data, the second is  
compressed data and rocksdb compress  it again.

But is doesn't matter.  Do I understand that right?

> flink rocksdb StreamCompressionDecorator not right
> --------------------------------------------------
>
>                 Key: FLINK-14159
>                 URL: https://issues.apache.org/jira/browse/FLINK-14159
>             Project: Flink
>          Issue Type: Bug
>          Components: Runtime / State Backends
>    Affects Versions: 1.9.0
>            Reporter: jackylau
>            Priority: Major
>             Fix For: 1.10.0
>
>
> I think the current flink rocksdb StreamCompressionDecorator is not right 
> when calling method 
> getCompressionDecorator(executionConfig) ,which defalut value is false.That 
> is to say, current compression is none.But I find rocksdb  using 
> {{options.compression}} to specify the compression to use. By default it is 
> Snappy, which you can see here 
> [https://github.com/facebook/rocksdb/wiki/RocksDB-Tuning-Guide]. And I use 
> rocksdb tool sstdump to find it is indeed snappy compression.
> So I think it should be return SnappyStreamCompressionDecorator.INSTANCE  
> rather than getCompressionDecorator( executionConfig) 
> Coud i commit a PR?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to