1fanwang commented on PR #51163: URL: https://github.com/apache/arrow/pull/51163#issuecomment-5623941440
Swept the rest of the Cython bindings for the same pattern. Passing `None` to every public module-level callable across the main modules turned up one reachable crash, `dataset.get_partition_keys`, which killed the interpreter with a bus error. Filed as #51293 and fixed in #51294. That PR also adds a guard test so the class stays covered: it passes `None` to each public callable in a child process and fails naming the offender if the interpreter dies by signal. Most typed parameters handle `None` deliberately, so it asserts that nothing crashes rather than requiring `not None` everywhere. -- 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]
