andygrove commented on PR #3237: URL: https://github.com/apache/datafusion-comet/pull/3237#issuecomment-3786696115
> How this will be maintained in the future ? Is it expected that the Iceberg maintainers and contributors will open a PR here every time they start using a method/class from Comet that is not yet annotated with `@IcebergApi`? Or to remove `@IcebergApi` when they stop using some annotated method ? Or it will be Comet's devs responsibility to "scan" Iceberg code for new/obsolete usage ? The next step is https://github.com/apache/datafusion-comet/pull/3240 which adds unit tests to ensure that the API is as expected. This will help prevent accidental changes to the current API methods. We should probably update the release process documentation to suggest auditing the Iceberg API use before creating a new release. -- 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]
