NebulousLabs / Sia

Blockchain-based marketplace for file storage. Project has moved to GitLab: https://gitlab.com/NebulousLabs/Sia
https://sia.tech
MIT License
2.71k stars 442 forks source link

Persist renter cache settings, and renter bandwidth limiter settings #3052

Closed MSevey closed 6 years ago

MSevey commented 6 years ago

This resolves #3043

MSevey commented 6 years ago

Need to replicate persistence struct from miner for renter. replace data struct in saveSync

DavidVorick commented 6 years ago

I could be mistaken, but it looks like (based on one of my comments) this PR introduced a bug where the stream cache was not being initialized correctly on startup, even though the user's stream cache size was being persisted correctly.

So we should probably extent the persistence test that you wrote to also check that the actual cache (instead of just the persisted value) is initialized correctly at startup.