jayzhan211 commented on PR #24519: URL: https://github.com/apache/datafusion/pull/24519#issuecomment-5425130735
> Yes, just a suggestion, not a hard requirement. It should be good to go if others can review, I might not be able to get to it timely. Thanks for the heads-up! That sounds good to me. I’ll ask others to take a look as well, and no worries if you don’t get a chance to review it in time. @alamb Could you take a look when you have a chance? This also looks good to @Xuanwo. I think we can merge it without a benchmark, similar to what we did for the physical operator in #23828, and continue iterating on it afterward. -- 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]
