XJDKC commented on PR #2759: URL: https://github.com/apache/polaris/pull/2759#issuecomment-3383438615
> Generally LGTM, just a comment about the naming of the `expiration-time` property. We could fixup in a followup if needed Good catch! I also looked into the Iceberg SDK, unfortunately, there isn't currently a property that indicates the expiration time of the token/credential for remote catalog access. So for now, I reused the `expiration-time` property, which is already used by `StorageAccessProperty`. I can modify this in a follow up PR! -- 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]
