leaves12138 commented on PR #8112:
URL: https://github.com/apache/paimon/pull/8112#issuecomment-4619905392

   Thanks for adding this endpoint. Two things should be addressed before merge:
   
   1. `DelegateCatalog#getTableVia` delegates directly to 
`wrapped.getTableVia(...)`. For wrappers like `PrivilegedCatalog`, this may 
bypass the `getTable` wrapping that returns `PrivilegedFileStoreTable`. Please 
override `getTableVia` there, or make the default path preserve the same 
privilege wrapping.
   2. Please update `docs/static/rest-catalog-open-api.yaml` for the new 
`/via/{viaDatabase}/{viaView}` endpoint.


-- 
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]

Reply via email to