kevinjqliu commented on issue #1589: URL: https://github.com/apache/iceberg-python/issues/1589#issuecomment-2622298632
Hey @bigluck good to hear from you again :) happy new year! You brought up an interesting use case and i think we should cement the right behavior. There are a couple of different sets of configs 1. catalog initialization configs, we can also call it "client config" since its the client of IRC 2. IRC server default (from the `/config` endpoint), this is the "catalog config" 3. namespace configs (the namespace itself can specify properties) 4. table configs From the server side, the table configs take precedence over namespace configs over server configs. Between the client and the server, I'm not sure which one should take precedence. Currently, the server configs take precedence and we override client configs during catalog initialization. https://github.com/apache/iceberg-python/blob/1adbb87627bfdfe80622d78c57b6214957520be0/pyiceberg/catalog/rest.py#L383-L402 We should figure out if this is the right behavior. Is the server config more of a suggestion or a must-override? -- 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...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
