CorfuDB icon indicating copy to clipboard operation
CorfuDB copied to clipboard

[WIP] MVCC PoC

Open Lujie1996 opened this issue 3 years ago • 2 comments

Introduced PersistentCorfuTable which is backed by Vavr HashMap. Added PersistentCorfuCompileProxy and PersistentVersionLockedObject. Added a simple put and get test on PersistentCorfuTable.

Overview

Description:

Why should this be merged:

Related issue(s) (if applicable): #

Checklist (Definition of Done):

  • [ ] There are no TODOs left in the code
  • [ ] Coding conventions (e.g. for logging, unit tests) have been followed
  • [ ] Change is covered by automated tests
  • [ ] Public API has Javadoc

Lujie1996 avatar Feb 16 '22 00:02 Lujie1996

Great work. I think the patch needs some refactoring and explicit documentation on the concurrency model.

Maithem avatar Jul 20 '22 23:07 Maithem

I also think we should find better naming schemes for all new classes. For example, persistent is an overloaded and it might be confusing to understand the context (secondary storage persistence, or in memory?)

Maithem avatar Jul 20 '22 23:07 Maithem

Close this one as the same change has been merged by #3387.

Lujie1996 avatar Oct 29 '22 00:10 Lujie1996