agriyakhetarpal commented on PR #47330: URL: https://github.com/apache/arrow/pull/47330#issuecomment-6824400605
> I've just updated unneeded testing submodule change on this PR (so it no longer moves backward from Nov 5a26d35 to June 9ff285c). LGTM! Ah, thanks a lot @tadeja! 🙈 --- I have two follow-ups here: - I found a bug on the Pyodide side that I am working to prepare a non-arrow reproducer on, which came from investigating `test_read_csv_gil_deadlock`, which is that a side module's weak symbols crash when running `time.sleep()`, when switching stacks with stubs generated from them beneath them. - The testing infrastructure here can rely on `pyodide venv`, which is much simpler. That makes things more developer-friendly; the `NodeDriver` can be removed entirely, since the interpreter itself uses Node.js. I tested it locally, and we can enable the `ORC` tests that have been skipped, since we can access the timezone database dir through the venv. -- 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]
