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

[Aptos VM] Reduce some unnecessary transaction metadata instantiation #12545

Merged
merged 1 commit into from
Mar 15, 2024

Conversation

junkil-park
Copy link
Contributor

@junkil-park junkil-park commented Mar 15, 2024

Description

Reduce the unnecessary transaction metadata instantiation

This PR resolves #12546

Reduce the unnecessary transaction metadata instantiation
@junkil-park junkil-park force-pushed the jpark/minimize_txn_metadata_instantiation branch from 5e0f73b to 2168c89 Compare March 15, 2024 05:22
@junkil-park junkil-park changed the title [Aptos VM] Reduce the occurrences of the transaction metadata instantiation [Aptos VM] Reduce some unnecessary transaction metadata instantiation Mar 15, 2024
@junkil-park junkil-park enabled auto-merge (squash) March 15, 2024 16:40

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

✅ Forge suite compat success on aptos-node-v1.9.5 ==> 2168c8927e52a88fa0800218c8da1007d1792bc4

Compatibility test results for aptos-node-v1.9.5 ==> 2168c8927e52a88fa0800218c8da1007d1792bc4 (PR)
1. Check liveness of validators at old version: aptos-node-v1.9.5
compatibility::simple-validator-upgrade::liveness-check : committed: 6903 txn/s, latency: 4798 ms, (p50: 4800 ms, p90: 7800 ms, p99: 8400 ms), latency samples: 241620
2. Upgrading first Validator to new version: 2168c8927e52a88fa0800218c8da1007d1792bc4
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 675 txn/s, latency: 34894 ms, (p50: 36600 ms, p90: 55100 ms, p99: 57800 ms), latency samples: 56040
3. Upgrading rest of first batch to new version: 2168c8927e52a88fa0800218c8da1007d1792bc4
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 364 txn/s, submitted: 517 txn/s, expired: 153 txn/s, latency: 42202 ms, (p50: 41700 ms, p90: 57300 ms, p99: 58400 ms), latency samples: 31709
4. upgrading second batch to new version: 2168c8927e52a88fa0800218c8da1007d1792bc4
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 3093 txn/s, latency: 9788 ms, (p50: 9700 ms, p90: 14700 ms, p99: 16000 ms), latency samples: 136100
5. check swarm health
Compatibility test for aptos-node-v1.9.5 ==> 2168c8927e52a88fa0800218c8da1007d1792bc4 passed
Test Ok

Copy link
Contributor

✅ Forge suite realistic_env_max_load success on 2168c8927e52a88fa0800218c8da1007d1792bc4

two traffics test: inner traffic : committed: 7233 txn/s, latency: 5418 ms, (p50: 5100 ms, p90: 6900 ms, p99: 10500 ms), latency samples: 3124980
two traffics test : committed: 100 txn/s, latency: 2032 ms, (p50: 1900 ms, p90: 2200 ms, p99: 6100 ms), latency samples: 1760
Latency breakdown for phase 0: ["QsBatchToPos: max: 0.227, avg: 0.204", "QsPosToProposal: max: 0.343, avg: 0.306", "ConsensusProposalToOrdered: max: 0.485, avg: 0.463", "ConsensusOrderedToCommit: max: 0.327, avg: 0.308", "ConsensusProposalToCommit: max: 0.787, avg: 0.771"]
Max round gap was 1 [limit 4] at version 1500724. Max no progress secs was 4.404866 [limit 15] at version 1500724.
Test Ok

@junkil-park junkil-park merged commit 0a5455b into main Mar 15, 2024
82 checks passed
@junkil-park junkil-park deleted the jpark/minimize_txn_metadata_instantiation branch March 15, 2024 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Aptos VM] Reduce some unnecessary transaction metadata instantiation
3 participants