Skip to content
This repository has been archived by the owner on Mar 12, 2022. It is now read-only.

Fix issues related to precaching #44

Open
guilhermelawless opened this issue Mar 10, 2021 · 0 comments
Open

Fix issues related to precaching #44

guilhermelawless opened this issue Mar 10, 2021 · 0 comments
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@guilhermelawless
Copy link
Owner

Having received multiple reports of unexpected issues, such as receiving wrong work, a commit 638f22c has been pushed which disables precaching by default. This means all work is calculated on demand.

Benefits of disabling precache:

  • No longer needing a connection to a Nano node
  • Profoundly reduced complexity
  • Avoiding calculating work that won't be used if the account is being used elsewhere

Disadvantages:

  • Added latency for incoming requests

Since on demand work has been served with less than a second of latency, I don't see an issue with disabling precaching until the work difficulty increases substantially.

I don't expect to find the time to investigate the root of the problem. I would advise a complete re-write of the server with testing in mind, as this one is not ready for it.

@guilhermelawless guilhermelawless added bug Something isn't working wontfix This will not be worked on labels Mar 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant