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

zoucao commented on FLINK-20546:
--------------------------------

hi [~jark],thanks for your reply, Firstly. Of course I am willing to take it.

> Misuse of the method in KafkaDynamicTableFactoryTest
> ----------------------------------------------------
>
>                 Key: FLINK-20546
>                 URL: https://issues.apache.org/jira/browse/FLINK-20546
>             Project: Flink
>          Issue Type: Improvement
>          Components: Connectors / Kafka
>    Affects Versions: 1.12.0
>            Reporter: zoucao
>            Priority: Major
>
> hi all, I am reading something about DynamicTable Source/Sink Factory 
> recently, and one thing let me confused. In `KafkaDynamicTableFactoryTest`, 
> `getBasicSourceOptions()`  and `getBasicSinkOptions()` are created for source 
> table and sink table, respectively. However, in the situation of validating 
> sink table, `getBasicSourceOptions()` is called, for example in method 
> `testSinkWithTopicListOrTopicPattern`、`testInvalidSinkSemantic` etc.  
> Although no exception is thrown, it also should be fixed. WDYT?



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

Reply via email to