IntersectMBO / IntersectMBO/cardano-ledger

Improve performance of computation of `sizeTxF` for `ShelleyTx` and `AlonzoTx`.

Open
#4,901 1 comment 0 reactions 1 assignee Claimed by @teodanciu View on GitHub
:credit_card: technical-debt performance serialization
Dominant language
Haskell
Stars
295
Forks
179
Avg merge
4d 7h
Merged PRs (30d)
29

Description

There should be no reason why `ShelleyTx` retains the whole of original bytes, because it can't do so in a block anyways. The only reason why it uses `MemoBytes` is for size calculation needed for `sizedTxF`. Remove usage of `MemoBytes` for `ShelleyTx`, thus avoiding keeping around bytes for . However in order for us to memoize the size needed for computing the fee we should add a field to `AlonzoTx` and `ShelleyTx`. This will require creation of `AlonzoTxRaw` and removal of `MemoBytes` for `ShelleyTx`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.