[Archmonger]

By submitting this pull request you agree that all contributions to this project are made under the MIT license.

Description

Added

• ReactPy Websocket will now decode messages via orjson resulting in an ~6% overall performance boost.
• Built-in asyncio event loops are now patched via nest_asyncio, resulting in an ~10% overall performance boost on Windows. This has no performance impact if you are running your webserver with uvloop.

Fixed

• Fix bug where REACTPY_WEBSOCKET_URL always generates a warning if unset.
• Fixed bug where assert f is self._write_fut would be raised within uvicorn on Windows when REACTPY_BACKHAUL_THREAD = True.
• Fixed bug where rendering behavior would be jittery with daphne on Windows when REACTPY_BACKHAUL_THREAD = True.

[Archmonger]

@rmorshea I'm going to merge this as a temporary hotfix, but I'm honestly kind of shocked using nest_asyncio fixes both assert f is self._write_fut exceptions on Uvicorn and also jittery rendering behavior Daphne.

My best guess is the way that the Windows ProactorEventLoop enqueues tasks is inherently not thread safe, and maybe nest_asyncio serializes this task queue? I'm even more shocked that this results in a overall performance improvement.

There's still one open bug: Some Windows environments have degraded performance when running via backhaul thread. I've actually experienced this once in the past, but I've been unable to replicate it.

Will need to do a deeper dive into this in the future. Given that the issue existed for me before but is now gone (same Python version), this is likely somehow related to CPython optimizations.