mbutrovich commented on PR #48: URL: https://github.com/apache/datafusion-iceberg/pull/48#issuecomment-6148332920
> If nobody else gets to it, I'm happy to look into how we can create a more elegant interface, as I agree with Gabotechs assessment. I've been a bit tunnel-visioned on just opening publicize functions for Ballista to hook into, and it's better to get this right at the start then refactor after the fact. I'm okay merging this as-is, as long as the discussion about the API is summarized in an issue that we can track and address before cutting the first 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]
