Airblader commented on pull request #16745:
URL: https://github.com/apache/flink/pull/16745#issuecomment-894963011


   > i think the patch don't need FLINK-22878,because the options of 
catalogTable is Map, and then we just need to convert map to hiveConfig
   
   But these "passthrough" options are options for the Hive connector, right? 
I'm not really familiar with the Hive connector itself. We eventually want to 
generate all connector documentation from the `ConfigOption`s since they should 
be the source of truth. If we introduce things that bypass the configuration 
mechanism, they wouldn't be covered by the documentation.


-- 
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