andygrove commented on issue #2921: URL: https://github.com/apache/datafusion-comet/issues/2921#issuecomment-3785878915
Thanks for all the feedback so far! I wanted to share a couple of updates in Comet related to this. I have two PRs in Comet that I am looking for reviews on. The first PR adds an `@IcebergApi` annotation to all Java classes/methods/fields that are currently referenced from the Iceberg main branch, and also adds documentation to the contributors guide. https://github.com/apache/datafusion-comet/pull/3237 The second PR builds on this and adds a new `iceberg-api` maven module containing dedicated unit tests for API stability. This PR is more consequential because it discusses deprecating/removing Comet's `native_comet scan` and how that relates to the Iceberg pubic API. https://github.com/apache/datafusion-comet/pull/3240 I am not an expert on the current integration, so it is possible that I may have misunderstood things. I would appreciate reviews from both the Iceberg and Comet communities! -- 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]
