thisisnic commented on PR #46667: URL: https://github.com/apache/arrow/pull/46667#issuecomment-2931942020
> > cc @nealrichardson - is this good to go as-is? > > As long as the new test fails without the fix and it passes with it, I guess so? I don't understand how passing the `mask` in `eval_tidy()` twice would make a difference, but if everything is working and tests cover it, I don't need to understand 🤠Cool, just checking there wasn't anything "obvious" I was missing. I'll approve then. Cheers @etiennebacher for fixing 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
