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

add feature 'enable_execution_duration_record' #6

Conversation

xudaquan2003
Copy link

add feature 'enable_execution_duration_record'

@anonymousGiga anonymousGiga self-requested a review October 17, 2023 09:29
@@ -1,5 +1,7 @@
mod listener;
mod sync_metrics;
mod util;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add #[cfg(feature = "enable_execution_duration_record")]


pub use listener::{MetricEvent, MetricEventsSender, MetricsListener};
pub(crate) use util::*;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add #[cfg(feature = "enable_execution_duration_record")]

@@ -0,0 +1,67 @@

#[cfg(feature = "enable_execution_duration_record")]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete

pub(crate) write_to_db: u64,
}

#[cfg(feature = "enable_execution_duration_record")]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete

@anonymousGiga anonymousGiga merged commit e26e1b0 into andy/debug/performance-dashboard Oct 18, 2023
5 of 18 checks passed
anonymousGiga added a commit that referenced this pull request Nov 9, 2023
update

update

update

update

add finish_after_execution_stage feature for reth node

update revm

update log for revm metric

some update for performance-dashboard

update for write_to_db log

Split the get_revm_metric_record function into two

add feature 'enable_execution_duration_record' (#6)

* add feature 'enable_execution_duration_record'

* set 'enable_execution_duration_record' feature for util mod

refactor: refactoring the code of get revm metric (#8)

Refactor: refactoring the code of get cachedb metric (#9)

add feature 'enable_tps_gas_record' (#7)

* add feature 'enable_tps_gas_record'

* format and rebase code

---------

Co-authored-by: anonymousGiga <[email protected]>

use minstant instead of TimeRecord (#10)

update for revm dependencies (#11)

Enable sload percentiles (#14)

support test oom th (#15)

add feature 'enable_db_speed_record' (#12)

* add feature 'enable_db_speed_record'

* Some minor refactoring

---------

Co-authored-by: anonymousGiga <[email protected]>

update revm dependencies for opcode gas metric (#16)

update revm dependencies

refactor opcode metric (#17)

close some log

update dependencies

update revm dependencies

refactor performance dashboard (#19)

* support performance dashboard
* delete obsolete code

---------

Co-authored-by: anonymousGiga <[email protected]>

add sload duration percentile print
xudaquan2003 pushed a commit that referenced this pull request Nov 10, 2023
update

update

update

update

add finish_after_execution_stage feature for reth node

update revm

update log for revm metric

some update for performance-dashboard

update for write_to_db log

Split the get_revm_metric_record function into two

add feature 'enable_execution_duration_record' (#6)

* add feature 'enable_execution_duration_record'

* set 'enable_execution_duration_record' feature for util mod

refactor: refactoring the code of get revm metric (#8)

Refactor: refactoring the code of get cachedb metric (#9)

add feature 'enable_tps_gas_record' (#7)

* add feature 'enable_tps_gas_record'

* format and rebase code

---------

Co-authored-by: anonymousGiga <[email protected]>

use minstant instead of TimeRecord (#10)

update for revm dependencies (#11)

Enable sload percentiles (#14)

support test oom th (#15)

add feature 'enable_db_speed_record' (#12)

* add feature 'enable_db_speed_record'

* Some minor refactoring

---------

Co-authored-by: anonymousGiga <[email protected]>

update revm dependencies for opcode gas metric (#16)

update revm dependencies

refactor opcode metric (#17)

close some log

update dependencies

update revm dependencies

refactor performance dashboard (#19)

* support performance dashboard
* delete obsolete code

---------

Co-authored-by: anonymousGiga <[email protected]>

add sload duration percentile print
xudaquan2003 pushed a commit that referenced this pull request Nov 10, 2023
update

update

update

update

add finish_after_execution_stage feature for reth node

update revm

update log for revm metric

some update for performance-dashboard

update for write_to_db log

Split the get_revm_metric_record function into two

add feature 'enable_execution_duration_record' (#6)

* add feature 'enable_execution_duration_record'

* set 'enable_execution_duration_record' feature for util mod

refactor: refactoring the code of get revm metric (#8)

Refactor: refactoring the code of get cachedb metric (#9)

add feature 'enable_tps_gas_record' (#7)

* add feature 'enable_tps_gas_record'

* format and rebase code

---------

Co-authored-by: anonymousGiga <[email protected]>

use minstant instead of TimeRecord (#10)

update for revm dependencies (#11)

Enable sload percentiles (#14)

support test oom th (#15)

add feature 'enable_db_speed_record' (#12)

* add feature 'enable_db_speed_record'

* Some minor refactoring

---------

Co-authored-by: anonymousGiga <[email protected]>

update revm dependencies for opcode gas metric (#16)

update revm dependencies

refactor opcode metric (#17)

close some log

update dependencies

update revm dependencies

refactor performance dashboard (#19)

* support performance dashboard
* delete obsolete code

---------

Co-authored-by: anonymousGiga <[email protected]>

add sload duration percentile print
xudaquan2003 pushed a commit that referenced this pull request Nov 14, 2023
update

update

update

update

add finish_after_execution_stage feature for reth node

update revm

update log for revm metric

some update for performance-dashboard

update for write_to_db log

Split the get_revm_metric_record function into two

add feature 'enable_execution_duration_record' (#6)

* add feature 'enable_execution_duration_record'

* set 'enable_execution_duration_record' feature for util mod

refactor: refactoring the code of get revm metric (#8)

Refactor: refactoring the code of get cachedb metric (#9)

add feature 'enable_tps_gas_record' (#7)

* add feature 'enable_tps_gas_record'

* format and rebase code

---------

Co-authored-by: anonymousGiga <[email protected]>

use minstant instead of TimeRecord (#10)

update for revm dependencies (#11)

Enable sload percentiles (#14)

support test oom th (#15)

add feature 'enable_db_speed_record' (#12)

* add feature 'enable_db_speed_record'

* Some minor refactoring

---------

Co-authored-by: anonymousGiga <[email protected]>

update revm dependencies for opcode gas metric (#16)

update revm dependencies

refactor opcode metric (#17)

close some log

update dependencies

update revm dependencies

refactor performance dashboard (#19)

* support performance dashboard
* delete obsolete code

---------

Co-authored-by: anonymousGiga <[email protected]>

add sload duration percentile print

modify cargo default config

add enable_opcode_metrics feature in EVMProcessor

fix bug of caculate total_duration
@anonymousGiga anonymousGiga deleted the xudq/debug/enable_execution_duration_record branch December 8, 2023 11:08
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