[Persistence] Fix Persistence Module TODOs - Multiple Write Contexts #149

[jessicadaugherty]

Objective

A general cleanup issue is needed to tackle TODO's and ensure the persistence module is usable/readable by handling multiple write contexts as part of #172.

Origin Document

Should follow issue-#128, issue-#105, issue-#147 and issue-#148 the persistence module is messier and more difficult to understand than the developers would want for organic external contribution.

Goals / Deliverables

• Handle multiple write contexts. See the TODO near the TestTwoWriteContextsSameHeight test in persistence/test/module_test.go.

General issue checklist

• Update the appropriate CHANGELOG
• Update the README
• If applicable, update the source code tree explanation
• If applicable, add or update a state, sequence or flowchart diagram using mermaid
• Update any relevant global documentation & references
• Document small issues / TODOs along the way

---

Creator: @andrewnguyen22
Co-creator: @Olshansk

[Olshansk]

@andrewnguyen22 Did we agree that there should only ever be one write context at any point in time? If so, I think we can close this out.

[Olshansk]

@andrewnguyen22 Bump on question above

[Olshansk]

Going to close this out given the current state of persistence, but lmk if it should be something we re-explore.