stevenzwu opened a new pull request #2742: URL: https://github.com/apache/iceberg/pull/2742
…e can centralize all configs (table, source, sink etc.) in this single class This is to address this comment: https://github.com/apache/iceberg/pull/2305#discussion_r591421143 The config class is only used even though it is a public class. To users, what matters are the config keys. They are the public contract and weren't changed. So it shouldn't cause any compatibility issue. cc @holdenk @openinx -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
