jerryshao opened a new issue, #11131: URL: https://github.com/apache/gravitino/issues/11131
## Describe the proposal A source-code review of the `iceberg-rest-server` module and supporting modules identified 8 issues across correctness, scalability, and maintainability. The underlying architecture is solid — credential cache with token-lifetime-aware expiry, metadata cache with multi-server-coherent pointer validation, ETag fast paths, atomic Caffeine cache loaders. The remaining issues are mostly defaults and gaps rather than missing capabilities. ## Task list - [ ] NamespaceNotEmptyException maps to HTTP 400 instead of 409 (spec non-conformance) - [ ] Table metadata cache disabled by default; default capacity (200) too small for enterprise scale - [ ] List endpoints (listTable, listNamespaces, listView) do not support pagination (pageToken/pageSize) - [ ] Credential cache expire_ratio default 0.15 generates ~5x more STS calls than necessary - [ ] RBAC OGNL expression parse and conversion repeat on every authorized request - [ ] No HTTP timeout configuration on RESTCatalog backend used for federation - [ ] registerTable does not follow the federation FileIO-property-extraction pattern that loadTable does - [ ] Federation behavior implemented as scattered instanceof RESTCatalog short-circuits in CatalogWrapperForREST -- 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]
