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

v13 migration: Audit and clean up the two-stage cache utilization #250

Open
arajasek opened this issue Feb 28, 2024 · 0 comments
Open

v13 migration: Audit and clean up the two-stage cache utilization #250

arajasek opened this issue Feb 28, 2024 · 0 comments

Comments

@arajasek
Copy link
Collaborator

Unlike previous migrations, the v13 migration has different behaviour depending on whether or not it's running in "cached" or "uncached" mode. This means that if, for some reason, the cache isn't correctly and completely populated in a previous migration ("pre-migration"), the result will be WRONG.

While this is still correct in the happy path, we need to robust-ify this. In particular, we should consider:

  • having some top-level directive about which "mode" to be running in (currently the individual miners and the market attempt to determine that themselves)
  • decoupling the miner and market's usages of the cache (currently it would be WRONG if a single miner ran in "un-cached" mode, but the market actor was migrated in cached mode).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

1 participant