[GitHub] [iceberg] hililiwei commented on pull request #5967: Flink: Support read options in flink source

[GitBox <gi...@apache.org>]

hililiwei commented on PR #5967:
URL: https://github.com/apache/iceberg/pull/5967#issuecomment-1333783891

   > > Hmm, I understand your concern now. The job-level configuration should have a connector prefix that makes sense to me, shall we consider using the same prefix?
   > 
   > Yeah. I was worry about config naming collision. A common prefix like `connector.iceberg.` should be enough. But it will be a problem to change it for the write configs, which is already released.
   
   I'm in favor of adding a prefix for the new one. The old ones are left as is for compatibility.
   
   
   
   


-- 
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-unsubscribe@iceberg.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org