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

david radley commented on FLINK-34211:
--------------------------------------

Hi [~xuyangzhong] , I have changed this to an improvement. Are you ok to assign 
this to me and merge the pr (with unit tests)? As the fix seems reasonably 
obvious; or do you think there are other considerations around this that 
warrant deeper discussion? As part of this I can add a tests for ? for scan 
queries.    

 

> Filtering on Column names with ?s fails for JDBC lookup join. 
> --------------------------------------------------------------
>
>                 Key: FLINK-34211
>                 URL: https://issues.apache.org/jira/browse/FLINK-34211
>             Project: Flink
>          Issue Type: Improvement
>          Components: Connectors / JDBC, Table SQL / JDBC
>            Reporter: david radley
>            Priority: Minor
>
> There is a check for ? character in 
> [https://github.com/apache/flink-connector-jdbc/blob/e3dd84160cd665ae17672da8b6e742e61a72a32d/flink-connector-jdbc/src/main/java/org/apache/flink/connector/jdbc/statement/FieldNamedPreparedStatementImpl.java#L186
>  |FieldNamedPreparedStatementImpl.java]
> Removing this check allows column names containing _?_ 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to