LiebingYu commented on issue #1890:
URL: https://github.com/apache/fluss/issues/1890#issuecomment-3466486125
> 1. Maybe `FlussPrincipal` can be wrapped in a `Context` in case in the
future we may add others for `createTable` method?
>
> ```
> Context {
> FlussPrincipal
> }
> ```
>
> 2. I feel like it a little of complex to introduce `authorize`,
`revokeAuthorization`, can `DLFPaimonCatalog` just over right `createTable`
method to authorize and then `createTable`? We can consider it introduce it
later if we do find need it. But I prefer not to do overdesign
1. `Context` is good to me.
2. Ok let's keep it simple now.
--
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]
