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

Fix in mem ds cache err handling cherry pick #12920

Merged
merged 3 commits into from
Apr 23, 2024

Conversation

ilija42
Copy link
Contributor

@ilija42 ilija42 commented Apr 22, 2024

No description provided.

ilija42 added 2 commits April 23, 2024 00:23
…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
@ilija42 ilija42 requested review from a team as code owners April 22, 2024 22:24
@@ -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]
Copy link
Contributor

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

Copy link
Contributor Author

@ilija42 ilija42 Apr 22, 2024

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

@ilija42 ilija42 changed the title Fix in mem ds cache err handling Fix in mem ds cache err handling cherry pick Apr 22, 2024
* Bumping solana

* Bumping the checksum for solana validator
@ilija42 ilija42 requested a review from krehermann April 23, 2024 15:00
@ilija42 ilija42 merged commit fcd1d3f into release/2.11.0 Apr 23, 2024
115 checks passed
@ilija42 ilija42 deleted the fix-in-mem-ds-cache-err-handling branch April 23, 2024 15:01
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.

3 participants