Closed dktapps closed 1 year ago
merging-members.phpt
now fails because of a bug in local cache handling, which is not caused by this PR, but now shows up in this test because strings are now retained in local caches.
Local testing indicates that this offers significant performance gains for channel-style use-cases seen in PM. In particular, since PM sends far more traffic out than it receives, this change offers a significant performance advantage to the main server thread.
It's unclear exactly how much real-world benefit this will have, but I'm very happy with the results so far.
This is an experimental implementation of #71.
If this works, it would offer some nice performance advantages to PM, since its inter-thread channels would benefit from this, as would async tasks.