Details
-
Improvement
-
Status: Open
-
Minor
-
Resolution: Unresolved
-
None
-
None
-
None
Description
- To be proper, minCommittedLog/maxCommittedLog should only be modified in a lock.
- maxCommittedLog is potentially set twice (to the same value) for each call to the method
- Streamline code in addCommittedProposal
- Pre-initialize committedLog data structure to the full size
- Remove unused commitLogBuffer
- Remove synchronization of method getCommittedLog(), it is protected with lock
- Unify grabbing locks outside of try block
- Fix off-by-one error in the size of the buffer
Attachments
Issue Links
- links to