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

Leonard Xu commented on FLINK-17826:
------------------------------------

Hi, [~f.pompermaier] 

The concern about the custom query is that 
(1) After add a custom query, the JDBC table should be a view of JDBC (e.g 
select * from db1.A join db1.B) instead of a table from semantics.

(2) The table may can only be used as source table and can not as sink table 
for example above case.

base on these,  I tend to do not add the unclear option.

But If you're interesting this issue, you can take over and I can help review 
and merge. We can add some specification  for (1) and add more checks for (2)

 

> Add missing custom query support on new jdbc connector
> ------------------------------------------------------
>
>                 Key: FLINK-17826
>                 URL: https://issues.apache.org/jira/browse/FLINK-17826
>             Project: Flink
>          Issue Type: Bug
>          Components: Connectors / JDBC
>            Reporter: Jark Wu
>            Assignee: Leonard Xu
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.13.0
>
>
> In FLINK-17361, we added custom query on JDBC tables, but missing to add the 
> same ability on new jdbc connector (i.e. 
> {{JdbcDynamicTableSourceSinkFactory}}). 
> In the new jdbc connector, maybe we should call it {{scan.query}} to keep 
> consistent with other scan options, besides we need to make {{"table-name"}} 
> optional, but add validation that "table-name" and "scan.query" shouldn't 
> both be empty, and "table-name" must not be empty when used as sink.



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

Reply via email to