Skip to content

Commit

Permalink
[TRIVIAL] Simplify estimator logs (#2772)
Browse files Browse the repository at this point in the history
# Changes
When quote verification was fresh it made sense to have special logging
for verified estimators to debug things. But now all estimators are
always verified and the `_verified` postfix makes it so that you don't
get to see all the logs in kibana when you just search for `<estimator>`
(`<estimator>_verified` logs would not show up).
  • Loading branch information
MartinquaXD authored Jun 6, 2024
1 parent 93519e1 commit baf9a1a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion crates/shared/src/price_estimation/factory.rs
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ impl<'a> PriceEstimatorFactory<'a> {

let fast = instrument(estimator, name);
let optimal = match verified {
Some(verified) => instrument(verified, format!("{name}_verified")),
Some(verified) => instrument(verified, name),
None => fast.clone(),
};

Expand Down

0 comments on commit baf9a1a

Please sign in to comment.