laserninja commented on PR #11617: URL: https://github.com/apache/gravitino/pull/11617#issuecomment-4697527785
True that the Iceberg REST spec currently defines ETag/If-None-Match only for loadTable, not for loadView. This PR extends the same pattern to views as a Gravitino-level enhancement. The rationale: 1. Views have the same metadataFileLocation property that tables use for ETag generation, so the mechanism is identical 2. ETag/If-None-Match is a standard HTTP caching mechanism (RFC 7232) that can apply to any resource, it's not table-specific semantics 3. It's fully backward-compatible, clients that don't send If-None-Match get the exact same response as before It reduces unnecessary data transfer for clients polling view metadata that hasn't changed That said, if the project prefers to stay strictly within the Iceberg REST spec for views, I understand. Let me know how you'd like to proceed, happy to adjust. -- 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]
