Conversation
Some other things came up but I'm going to come back to this next week. |
|
I ran the test suite with I see some tracebacks inside All that to say, I think I agree with #1295 (comment) that we should focus on multithreaded stress tests rather than trying to use the existing test suite as a stress test - we'll likely end up spending most of our effort on fixing races in the test suite itself. For now I'll focus on writing a stress test that uses a thread pool to process many files simultaneously. |
|
I doubt it'll be very elucidating (it's 1.5 MB...) but here's the full output from running |
d6c78bd to
d838c82
Compare
dbd34c2 to
404bef2
Compare
|
Nice! I updated the tracking table too: Quansight-Labs/free-threaded-compatibility#215 |
This PR: 1. marks the `libcst.native` module as free-threading-compatible 2. replaces the use of ProcessPoolExecutor with ThreadPoolExecutor if free-threaded CPython is detected at runtime
Enable support for free-threading
This PR:
libcst.nativemodule as free-threading-compatibleThis depends on #1294 and #1289.
Stack created with Sapling. Best reviewed with ReviewStack.