Skip to content

Move casting protocol into rust #851

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 8 commits into from

Conversation

zdevito
Copy link
Contributor

@zdevito zdevito commented Aug 13, 2025

Stack from ghstack (oldest at bottom):

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: D80037834

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)

[ghstack-poisoned]
zdevito added a commit that referenced this pull request Aug 13, 2025
This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)

ghstack-source-id: 302731839
Pull Request resolved: #851
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Aug 13, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D80037834

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)

[ghstack-poisoned]
zdevito added a commit that referenced this pull request Aug 13, 2025
Pull Request resolved: #851

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.
ghstack-source-id: 302743564

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D80037834

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)

[ghstack-poisoned]
zdevito added a commit that referenced this pull request Aug 13, 2025
Pull Request resolved: #851

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.
ghstack-source-id: 302748010

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D80037834

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)

[ghstack-poisoned]
zdevito added a commit that referenced this pull request Aug 13, 2025
Pull Request resolved: #851

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.
ghstack-source-id: 302759496

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D80037834

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)

[ghstack-poisoned]
zdevito added a commit that referenced this pull request Aug 13, 2025
Pull Request resolved: #851

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.
ghstack-source-id: 302760277

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D80037834

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)

[ghstack-poisoned]
zdevito added a commit that referenced this pull request Aug 14, 2025
Pull Request resolved: #851

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.
ghstack-source-id: 303021421

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D80037834

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)

[ghstack-poisoned]
zdevito added a commit that referenced this pull request Aug 14, 2025
Pull Request resolved: #851

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.
ghstack-source-id: 303110512

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D80037834

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)

[ghstack-poisoned]
zdevito added a commit that referenced this pull request Aug 15, 2025
Pull Request resolved: #851

This moves the logic for deferring sends into Rust to avoid slowing down actor sends. Increases send throughput ~30% from the python approach. It is just a little faster than what came before the python approach because this also eliminates a layer of python wrappers for dispatching sends. I suspect there is tons more to to be gained removing more layers of python between the user calling an endpoint and issuing a send.
ghstack-source-id: 303184134

Differential Revision: [D80037834](https://our.internmc.facebook.com/intern/diff/D80037834/)
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D80037834

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in cce99fb.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants