Skip to content

Python actor call latency #801

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pablorfb-meta
Copy link
Contributor

Summary:
Measures how long it takes to cast and reply to 1Kb to N hosts with 8 actors via local transport
100ms of processing time (included in benchmark below) was added to the tested endpoint

Benchmark Name Median Latency (ms) Median Latency Rust (ms) Difference (%) (Python vs Rust)
actor_count_1_median_ms 108 101.55 6.35%
actor_count_10_median_ms 112 101.78 10.04%
actor_count_100_median_ms 145 102.98 40.81%
  • Crashes on 1k "host" set up

Differential Revision: D79769681

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Aug 8, 2025
@facebook-github-bot
Copy link
Contributor

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

Summary:
Measures how long it takes to cast and reply to 1Kb to N hosts with 8 actors via local transport
100ms of processing time (included in benchmark below) was added to the tested endpoint

| Benchmark Name           | Median Latency (ms) for Python | Median Latency Rust (ms) | % Diff |  % Diff w/o proc time|
|-------------------------|-------------------------------|-------------------------|---------------------------------|-----------------------|
| actor_count_1_median_ms  | 108                           | 101.55                  | 6.35%                           | 516%                  |
| actor_count_10_median_ms | 112                           | 101.78                  | 10.04%                          | 674%                  |
| actor_count_100_median_ms| 145                           | 102.98                  | 40.81%                          | 1510%                 |

* Crashes on 1k "host" set up

Differential Revision: D79769681
@facebook-github-bot
Copy link
Contributor

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

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants