FANNG1 commented on issue #8311:
URL: https://github.com/apache/gravitino/issues/8311#issuecomment-3310155052

   > > [@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. 


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