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

Track transaction performance track txn in banking stage deep #35402

Conversation

lijunwangs
Copy link
Contributor

Problem

test only -- please ignore

Summary of Changes

Fixes #

trying txn mask matching

output txn to figure out why txn is not exactly matched

Use 62 and 61 portion

track fetch performance using random txn mask

track sigverify performance using random txn mask

track banking stage performance using random txn mask

adding missing cargo lock file

add debug messages

Revert "add debug messages"

This reverts commit 96aead5.

fixed some clippy issues

check-crate issue

Fix a clippy issue

Fix a clippy issue

debug why txns in banking stage shows fewer performance tracking points

debug why txns in banking stage shows fewer performance tracking points

debug why txns in banking stage shows fewer performance tracking points

debug why txns in banking stage shows fewer performance tracking points

get higher PPS for testing purpose

more debug messages on why txn is skipped

display if tracer packet in log

add debug before calling processing_function

debug at the initial of banking stage

track if a txn is forwarded

dependency order

missing cargo file

clean up debug messages

Do not use TRACER_PACKET, use its own bit

rename some functions

addressed some comments from Trent

Update core/src/banking_stage/immutable_deserialized_packet.rs

Co-authored-by: Trent Nelson <[email protected]>

addressed some comments from Trent

Do not use binary_search, do simple compare in one loop
@lijunwangs lijunwangs force-pushed the track_transaction_performance_track_txn_in_banking_stage_deep branch from 6ace7b5 to f23f506 Compare March 2, 2024 11:29
@willhickey
Copy link
Contributor

This repository is no longer in use. Please re-open this pull request in the agave repo: https://github.com/anza-xyz/agave

@willhickey willhickey closed this Mar 3, 2024
@lijunwangs
Copy link
Contributor Author

This repository is no longer in use. Please re-open this pull request in the agave repo: https://github.com/anza-xyz/agave

Can we retain the review history, when we reopen?

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.

2 participants