xuzhiwen1255 commented on PR #28:
URL: 
https://github.com/apache/flink-connector-jdbc/pull/28#issuecomment-1507811739

   @MartijnVisser Sorry, i made a mistake, it should be that the current jdbc 
lookup configuration is obsolete, and we should replace it with the latest 
lookup configuration.
   
   eg. Configure ``lookup.cache.max-rows`` to replace 
``lookup.partial-cache.max-rows`` ..
   
   Otherwise, when users configure according to the official website 
information, they are still using the old configuration parameters instead of 
the latest ones.
   I think we should modify all obsolete parameters and replace them with the 
latest lookup configuration.
   
   What do you think? @MartijnVisser 


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to