sunxiaojian commented on issue #8311: URL: https://github.com/apache/gravitino/issues/8311#issuecomment-3310361604
> > > [@FANNG1](https://github.com/FANNG1) Thanks for the reminder. If there's already an issue implementing this, let's wait for the Iceberg PR to be merged before integrating. There's no need to re-implement it ourselves. > > > > > > [@FANNG1](https://github.com/FANNG1) If we implement it ourselves, there are mainly two points: > > > > * **server-side**: We can implement according to the endpoint specification of the Iceberg catalog, since Iceberg has already provided the open API definition for the scan plan. > > * **client-side**: We should need to override `RESTSessionCatalog` to adapt the logic for the scan plan. > > > > WDYT? > > We could implement the server-side first, we could do a simple test by constructing the rest request. For the client side, I prefer to cooperate with the upstream Iceberg community to do the work. Ok, I agree with you. -- 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]
