Skip to content
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

[optqs] support fetching batches from QC signers #15452

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

ibalajiarun
Copy link
Contributor

Description

Copy link

trunk-io bot commented Dec 2, 2024

⏱️ 4h 18m total CI duration on this PR
Slowest 15 Jobs Cumulative Duration Recent Runs
test-target-determinator 53m 🟩🟩🟩🟩🟩 (+8 more)
rust-cargo-deny 26m 🟩🟩🟩🟩🟩 (+10 more)
check-dynamic-deps 18m 🟩🟩🟩🟩🟩 (+11 more)
rust-move-tests 14m 🟩
rust-move-tests 13m 🟩
rust-move-tests 13m 🟩
rust-move-tests 13m 🟩
rust-move-tests 13m
rust-move-tests 13m 🟩
rust-move-tests 12m 🟩
rust-move-tests 12m 🟩
rust-move-tests 10m
rust-move-tests 10m
rust-move-tests 8m 🟥
semgrep/ci 7m 🟩🟩🟩🟩🟩 (+11 more)

settingsfeedbackdocs ⋅ learn more about trunk.io

@ibalajiarun ibalajiarun force-pushed the balaji/optqs-batch branch 2 times, most recently from 4f486f1 to b55b5f0 Compare December 2, 2024 23:22
@ibalajiarun ibalajiarun added the CICD:run-forge-e2e-perf Run the e2e perf forge only label Dec 2, 2024

This comment has been minimized.

@ibalajiarun ibalajiarun changed the title Balaji/optqs batch [optqs] support fetching batches from QC signers Dec 2, 2024

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@ibalajiarun ibalajiarun marked this pull request as ready for review December 3, 2024 01:45

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@ibalajiarun ibalajiarun force-pushed the balaji/optqs-batch branch 2 times, most recently from 3dab484 to be1a1b2 Compare December 3, 2024 19:12

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

github-actions bot commented Dec 3, 2024

✅ Forge suite realistic_env_max_load success on ab1b12089a435bfbcf6659fea591625aa9a0bc15

two traffics test: inner traffic : committed: 14216.02 txn/s, latency: 2915.01 ms, (p50: 2700 ms, p70: 2700, p90: 3000 ms, p99: 17500 ms), latency samples: 5405300
two traffics test : committed: 100.06 txn/s, latency: 2348.54 ms, (p50: 1400 ms, p70: 2000, p90: 3000 ms, p99: 16400 ms), latency samples: 1800
Latency breakdown for phase 0: ["MempoolToBlockCreation: max: 2.041, avg: 1.338", "ConsensusProposalToOrdered: max: 0.326, avg: 0.294", "ConsensusOrderedToCommit: max: 0.387, avg: 0.376", "ConsensusProposalToCommit: max: 0.680, avg: 0.670"]
Max non-epoch-change gap was: 1 rounds at version 18712 (avg 0.00) [limit 4], 2.50s no progress at version 18712 (avg 0.21s) [limit 15].
Max epoch-change gap was: 0 rounds at version 0 (avg 0.00) [limit 4], 15.67s no progress at version 5818058 (avg 14.52s) [limit 16].
Test Ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CICD:run-forge-e2e-perf Run the e2e perf forge only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant