-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Fix in mem ds cache err handling cherry pick #12920
Conversation
…12595) * Add jfpc cache staleness alert to config and pass it into jfpc cache * Extract JuelsPerFeeCoinCache cfg into a separate struct, improve naming * Changeset * Update config validation test * Reduce changeset to minor
…cache (#12907) * Fix inMemoryDataSourceCache get() warn log formatting * Fix in mem ds cache updateCache() to save runs even if some ds failed * Improve in memory data source cache Observe() logs and error messages * Add changeset
@@ -2189,7 +2190,8 @@ ds1_multiply [type=multiply times=1.23]; | |||
ds1 -> ds1_parse -> ds1_multiply -> answer1; | |||
answer1 [type=median index=0]; | |||
""" | |||
juelsPerFeeCoinCacheDuration = "30s" | |||
[pluginConfig.juelsPerFeeCoinCache] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this going to break existing specs? if not, why not - it seems like a parsing change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We never distributed specs that have this set, default value is always used
* Bumping solana * Bumping the checksum for solana validator
Quality Gate passedIssues Measures |
No description provided.