feat: enable direct building of levelcache stores #72
Conversation
|
@dignifiedquire If this is urgent, it works and can be merged. Otherwise, I plan to add more tests and cleanups, etc. Let me know if this PR should also include the removal of the v1/v2 compaction, etc as well like we discussed. As-is, everything is backward compatible in terms of levelcache usage. |
cryptonemo
force-pushed
the
levelcache-direct
branch
2 times, most recently
from
c96e9c0
to
f6cde56
Compare
|
@cryptonemo if we want to make breaking changes in that front, we should get them in soon, likely sometime this week. No rush on getting this merged |
|
@dignifiedquire Not sure if this is what you're saying or opposing, but at this point, I think it's better to merge this separately, since it could be integrated/tested in rust-fil-proofs over time without impacting much else. At some later point when that later work is merged and compaction is no longer used, it makes sense to rip that all out then. |
|
Thanks, I'll merge this shortly. In the meantime, I've started testing out the integration and want to make sure all is well first. It's coming together, but not quite there yet. At the moment, I don't anticipate changes required here. Just being cautious ;-) |
cryptonemo
force-pushed
the
levelcache-direct
branch
from
6289b99
to
05a45b5
Compare
|
@dignifiedquire Made a very minor cosmetic update. Will merge when CI is complete/happy. |
No description provided.