jbonofre commented on PR #10256: URL: https://github.com/apache/iceberg/pull/10256#issuecomment-2110987236
Maybe the confusion is due to the `config` endpoint is mixing different semantic properties, also related to the `scope`. For clarity, it would be better to split non authenticated config (`handshake`) from authenticated one (`config`), that's part of the "new" REST Catalog proposal. I understand @snazy points, but related to the current implementation (as @danielcweeks said), it's confusing. If I may, I would propose to: 1. the current impl should be improved, I think @adutra change is OK if we add scope configurable to deal with server side push 2. this change (dual config endpoints, auth and non auth) will be part of the "new" REST Catalog proposal Thoughts ? -- 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
