NoahKusaba commented on PR #48: URL: https://github.com/apache/datafusion-iceberg/pull/48#issuecomment-6148192800
> There both seems to be eagerness to merge this, and concern about public API. > > > I don't love the fact that the public API of this project makes incorrect state representable. > > Is it in scope for this PR, or do we need to file issues to follow up? I don't want this to get lost and am catching up on PRs on this repo. 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. -- 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]
