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

Timo Walther edited comment on FLINK-25390 at 1/3/22, 12:11 PM:
----------------------------------------------------------------

Good point. `Factory` might be too generic but yes we can introduce a similar 
option for `DecodingFormat` and `EncodingFormat`. But we have to think this 
though because from the current method signature we cannot derive the format 
for discovery. I guess we have to discuss the signature again here. An 
alternative could also be that a format is always mutable.


was (Author: twalthr):
Good point. `Factory` might be too generic but yes we can introduce a similar 
option for `DecodingFormat` and `EncodingFormat`. But we have to think this 
though because from the current method signature we cannot derive the format 
for discovery. I guess we have to discuss the signature again here.

> Support mutable options in DynamicTableFactory
> ----------------------------------------------
>
>                 Key: FLINK-25390
>                 URL: https://issues.apache.org/jira/browse/FLINK-25390
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table SQL / API
>            Reporter: Timo Walther
>            Assignee: Francesco Guardiani
>            Priority: Major
>
> Factory update for mutable options.
> {code}
> DynamicTableFactory {
>     /* Declares options that can be modified without runtime implications. */
>     Set<ConfigOption<?>> mutableOptions();
>     
> }
> {code}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to