Rich-T-kid commented on PR #24271: URL: https://github.com/apache/datafusion/pull/24271#issuecomment-5329679772
> > make sense to me. are there any other operators that incorrectly account for their children's compute time? > > I haven't had a chance to audit the other operators for the same issue, so I can't say for sure yet. Given that, would you agree to roll the fix out progressively rather than blocking on a full audit, starting with this PR ? I can open a follow-up issue to track checking the other operators for the same compute-time accounting bug. @ulrichurriola-parada-ops I think it makes sense to create an issue explainging your findings and then create focused PR's. I'd be happy to review them feel free to ping me -- 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]
