-
Notifications
You must be signed in to change notification settings - Fork 23
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
3 changed files
with
82 additions
and
0 deletions.
There are no files selected for viewing
21 changes: 21 additions & 0 deletions
21
...nsensus-cardano/changelog.d/20240711_064942_nick.frisby_consolidate_txlimits.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
<!-- | ||
A new scriv changelog fragment. | ||
Uncomment the section that is right (remove the HTML comment wrapper). | ||
--> | ||
|
||
### Patch | ||
|
||
- Updates for the `TxLimits` mempool consolidation. | ||
|
||
### Non-Breaking | ||
|
||
- Do not check transaction sizes in the forging functions; simply include all | ||
given transactions. | ||
|
||
<!-- | ||
### Breaking | ||
- A bullet item for the Breaking category. | ||
--> |
23 changes: 23 additions & 0 deletions
23
...ensus-diffusion/changelog.d/20240711_064938_nick.frisby_consolidate_txlimits.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
<!-- | ||
A new scriv changelog fragment. | ||
Uncomment the section that is right (remove the HTML comment wrapper). | ||
--> | ||
|
||
|
||
### Patch | ||
|
||
- Updates for the `TxLimits` mempool consolidation. | ||
|
||
<!-- | ||
### Non-Breaking | ||
- A bullet item for the Non-Breaking category. | ||
--> | ||
<!-- | ||
### Breaking | ||
- A bullet item for the Breaking category. | ||
--> |
38 changes: 38 additions & 0 deletions
38
...boros-consensus/changelog.d/20240711_064934_nick.frisby_consolidate_txlimits.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,38 @@ | ||
<!-- | ||
A new scriv changelog fragment. | ||
Uncomment the section that is right (remove the HTML comment wrapper). | ||
--> | ||
|
||
<!-- | ||
### Patch | ||
- A bullet item for the Patch category. | ||
--> | ||
<!-- | ||
### Non-Breaking | ||
- A bullet item for the Non-Breaking category. | ||
--> | ||
|
||
### Breaking | ||
|
||
- Consolidate `TxLimits` in the mempool. | ||
- Remove `Mempool.`getTxSize`; the snapshot interface contains byte sizes | ||
now. | ||
|
||
- Transaction size, block capacity, and mempool capacity are | ||
multi-dimensional vectors (`ExUnits`, etc), instead of merely bytes: | ||
`TxMeasure`. | ||
|
||
- A transaction cannot be added if it would push any component of the size | ||
over that component of the mempool capacity. | ||
|
||
- The mempool capacity override is still specified in terms of bytes, but | ||
the magnitude is interpreted via division as a block count, rounded up. | ||
|
||
- Pass a correctly-sized prefix of the mempool to the forging functions, | ||
instead of its entire contents. The mempool's finger tree is best way to find | ||
that cutoff. |