sungwy commented on PR #18254: URL: https://github.com/apache/iceberg/pull/18254#issuecomment-6427395414
> @sungwy this is looking pretty good. Did you want to move this out of draft and start a discuss/vote series? Thank you for the review @danielcweeks - I will open this up for review, and start a DISCUSS thread on the dev mailing list. I have a mild preference for merging this spec change after https://github.com/apache/iceberg/pull/18080, as [this section](https://github.com/apache/iceberg/pull/18080/changes#r4361418141) describes how `X-Iceberg-Access-Delegation` is handled by signing endpoints. If we decide that additional language is not needed for handling `X-Iceberg-Access-Delegation` on `/sign` and `/sign/batch` endpoints, I agree that the order does not really matter. -- 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]
