davidradl commented on PR #79:
URL: 
https://github.com/apache/flink-connector-jdbc/pull/79#issuecomment-1895544871

   @libenchao cloned your repo. I like the idea behind this fix, avoiding 
String manipulation.  I added in all the table plan junits from the pr 79 and 
they work. The junit testing the FieldNamedPreparedStatement with a `:` in the 
column name fails.  
   
   I then ran local tests that work with pr 79. The lookup joins hang with the 
POC for me . I am using mySQL JDBC driver. I am continuing to investigate.
   
   
   
    


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to