NoahKusaba commented on PR #2613: URL: https://github.com/apache/iceberg-rust/pull/2613#issuecomment-4737679461
> Thanks @NoahKusaba for this pr, but I don't think it's the right direction to maintain such a huge integration in this repo. Currently the review resources is quite limited, and most committers are not familiar with ballista. I think it would be better to put it in ballista repo This still seems like the logical place to put it. This integration directly builds on the datafusion-iceberg integration that lives in this repo to interact with Iceberg. There already seems precedence for that with the Playground integration that lives in /integrations, also seems like other integrations with iceberg are parked here such as cache-moka. (still tightening up the implementation). -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
