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

Sivaprasanna Sethuraman commented on FLINK-16447:
-------------------------------------------------

Hi [~eskabetxe]

I don't think it is not a clean solution to extend a CompressionCodec just to 
implement the Serializable interface. We then have to document it somewhere so 
that the people who use it are aware. However, I agree with the point you made 
in my my PR ([#11307|https://github.com/apache/flink/pull/11307]) about the way 
codec extension is getting handled.  I have now updated my PR with the needed 
change.

[~kkl0u]

I think we are good now. Can we mark this as a duplicate?

> Non serializable field on CompressWriterFactory
> -----------------------------------------------
>
>                 Key: FLINK-16447
>                 URL: https://issues.apache.org/jira/browse/FLINK-16447
>             Project: Flink
>          Issue Type: Bug
>          Components: Formats (JSON, Avro, Parquet, ORC, SequenceFile)
>    Affects Versions: 1.10.0
>            Reporter: João Boto
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> CompressWriterFactory has a CompressionCodec that is not serializable..
> this make that StreamingFileSink fails to with non serializable field.
>  
> extending codec and implementing serializable solves the problem, but its odd



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

Reply via email to