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