laserninja commented on PR #10641: URL: https://github.com/apache/gravitino/pull/10641#issuecomment-4173685184
1. **Moved ETag helpers to `IcebergRESTUtils`**: `generateETag()` and `buildResponseWithETag()` now live in `IcebergRESTUtils` as public static methods. `IcebergTableOperations` delegates to them via private wrappers. `IcebergNamespaceOperations` calls `IcebergRESTUtils.buildResponseWithETag()` directly. No more cross-class package-private access. 2. **Added ETag value assertion**: `testRegisterTableReturnsETag` now computes the expected SHA-256 from the known mock metadata location (`/mock/metadata/v1.metadata.json` + `"all"`) and asserts the exact ETag value matches. -- 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]
