gabotechs commented on PR #48: URL: https://github.com/apache/datafusion-iceberg/pull/48#issuecomment-6149010990
I'm fine with merging this as-is now if that helps people get unblocked, as I unfortunately do not have a better suggestion of how to structure the API. Exposing a clean API will likely will likely have a domino effect in which we'll need to change the fields of several `struct`s in the project. I think it's worth exploring that and getting to the right structure will likely require some hands-on experimentation, but again, don't want to block other efforts so I'm fine with merging this as-is (probably with some issue created for tracking this). -- 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]
