Use Error log level as default

aptos-labs · Dec 4, 2024 · 290e210 · 290e210
1 parent 8f43629
commit 290e210
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/aptos-move/aptos-replay-benchmark/src/block.rs b/aptos-move/aptos-replay-benchmark/src/block.rs
@@ -6,7 +6,7 @@ use crate::{
     state_view::{ReadSet, ReadSetCapturingStateView},
     workload::Workload,
 };
-use aptos_logger::warn;
+use aptos_logger::error;
 use aptos_types::{
     block_executor::config::{
         BlockExecutorConfig, BlockExecutorConfigFromOnchain, BlockExecutorLocalConfig,
@@ -90,7 +90,7 @@ impl Block {
             {
                 for (state_key, _) in on_chain_output.write_set() {
                     if state_override.contains_key(state_key) {
-                        warn!(
+                        error!(
                             "Transaction {} writes to overridden state value for {:?}",
                             begin + idx as Version,
                             state_key

diff --git a/aptos-move/aptos-replay-benchmark/src/main.rs b/aptos-move/aptos-replay-benchmark/src/main.rs
@@ -33,7 +33,7 @@ feature. It takes two measurements: when Block-STM uses 2 threads, or 4 threads
 #[derive(Parser)]
 #[command(about, long_about = Some(LONG_ABOUT))]
 pub struct Command {
-    #[clap(long, default_value_t = Level::Warn)]
+    #[clap(long, default_value_t = Level::Error)]
     log_level: Level,
 
     #[clap(

diff --git a/aptos-move/aptos-replay-benchmark/src/overrides.rs b/aptos-move/aptos-replay-benchmark/src/overrides.rs
@@ -5,7 +5,7 @@
 //! transactions can be replayed on top of a modified state, and we can evaluate how it impacts
 //! performance or other things.
 
-use aptos_logger::warn;
+use aptos_logger::error;
 use aptos_types::{
     on_chain_config::{FeatureFlag, Features, OnChainConfig},
     state_store::{state_key::StateKey, state_value::StateValue, StateView},
@@ -38,13 +38,13 @@ impl OverrideConfig {
             config_override::<Features, _>(state_view, |features| {
                 for feature in &self.enable_features {
                     if features.is_enabled(*feature) {
-                        warn!("Feature {:?} is already enabled", feature);
+                        error!("Feature {:?} is already enabled", feature);
                     }
                     features.enable(*feature);
                 }
                 for feature in &self.disable_features {
                     if !features.is_enabled(*feature) {
-                        warn!("Feature {:?} is already disabled", feature);
+                        error!("Feature {:?} is already disabled", feature);
                     }
                     features.disable(*feature);
                 }