Faster ledger state

[berewt]

Several memory/speed improvement:

• Store volatile ledger states in memory (faster rollback)
• Use Builder instead of Bytestring for serialisation
• Save LedgerState less often, and always save it on close.
• Add a --batch-size flag

Pre-submit checklist:

• Branch
  • Tests are provided (if possible)
  • Commit sequence broadly makes sense and have useful messages
  • Important changes are reflected in changelog.d of the affected packages
  • Relevant tickets are mentioned in commit messages
• PR
  • (For external contributions) Corresponding issue exists and is linked in the description
  • Targeting main unless this is a cherry-pick backport
  • Self-reviewed the diff
  • Useful pull request description
  • If relevant, reference the ADR in the PR and reference the PR in the ADR
  • Reviewer requested

[brendanrbrown]

This all looks good. My comments are minor and mostly about documentation, particularly in cases where it's unclear that bad values of certain parameters are possible.

The CI currently fails because of some fourmolu thing.

But the tests failed for me locally because of golden files: I went ahead and fixed those up as in #286, in case you were still having issues with whitespace.

[berewt]

This all looks good. My comments are minor and mostly about documentation, particularly in cases where it's unclear that bad values of certain parameters are possible.

The CI currently fails because of some fourmolu thing.

But the tests failed for me locally because of golden files: I went ahead and fixed those up as in #286, in case you were still having issues with whitespace.

I still have these issues (I think I was able to do it in the past though, which is weird).

[brendanrbrown]

I still have these issues

Let me know if you want me to push the whitespace fix again.