mbutrovich commented on PR #20: URL: https://github.com/apache/datafusion-iceberg/pull/20#issuecomment-5894944631
> I also wanted to thank you for all of the time you've put into my many PR's I've made trying to land the Ballista integration, and I appreciate the care and precision you're putting into catching the stray bits of slop that slip past me. Thank you for the contributions! > I additionally noticed you took the time to review my forgotten PR's on iceberg-rust (thanks btw), so I wanted to note that I have at least 3-4 PR's I want to land in this repo (datafusion-iceberg). I have avoided opening more PR's, to be considerate to the maintainers, but I'm wondering if that would be preferable as you are an approving reviewer here? I think the DataFusion community is pretty comfortable moving quickly on this repo, which is part of the reason the code moved out of Iceberg for the table provider. I would say keep firing off reviews, but anything you can do to signal ordering is helpful. PRs that aren't your highest priority could sit as draft until their dependencies merge, and EPIC issues track stacked PRs/larger tasks well. > Just to note, I also have at least 5 other PR's I want to open on iceberg-rust, as well. That repo is a little trickier to manage, but we're trying over there, especially as it pertains to DataFusion-related PRs. Tag them with the `DataFusion` label, ping me or @comphead to try to review, and when things are ready I'll try to get the attention of a committer like @kevinjqliu for a review. Iceberg committers are pulled in a lot of directions, so we're trying to absorb some of the earlier review cycles. Thanks again! -- 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]
