AlenkaF commented on PR #51294: URL: https://github.com/apache/arrow/pull/51294#issuecomment-5630600617
Thanks for the PR. > Since a hand-audit goes stale, this adds two guard tests that pass None to public callables in a child process and fail naming the offender if the interpreter dies. One covers module-level functions, the other methods reached from a live object. The method sweep found Expression.equals after the function sweep came back clean. I am not sure this is needed. @raul what do you think? I would rather see a missing test for the `Expression.equals(None)` to be added. -- 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]
