Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pull in Consensys/tuweni release #8330

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

siladu
Copy link
Contributor

@siladu siladu commented Feb 19, 2025

Includes:

Needs testing

Includes:
- reduce check argument args allocations - Consensys/tuweni#10
which should reduce Integer and int[] and therefore gc pressure

Signed-off-by: Simon Dudley <[email protected]>
Signed-off-by: Simon Dudley <[email protected]>
CHANGELOG.md Outdated
@@ -22,6 +22,9 @@
- Allow plugins to propose transactions during block creation [#8268](https://github.com/hyperledger/besu/pull/8268)
- Update `eth_getLogs` to return a `Block not found` error when the requested block is not found. [#8290](https://github.com/hyperledger/besu/pull/8290)
- Improve Conflict Detection in Parallelization by Considering Slots to Reduce False Positives. [#7923](https://github.com/hyperledger/besu/pull/7923)
- Replace tuweni libs with https://github.com/Consensys/tuweni
- Performance: Consensys/tuweni 25.2.0 includes a signicant improvement that reduces memory allocations across Besu
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Performance: Consensys/tuweni 25.2.0 includes a signicant improvement that reduces memory allocations across Besu
- Performance: Consensys/tuweni 25.2.0 includes a significant improvement that reduces memory allocations across Besu

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would rather focus on the performance gain here, I mean latency and not memory allocations, something like "Reduce boxing/unboxing overhead on some EVM opcodes, like PushX and Caller".

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed to:

  • Performance: Consensys/tuweni 2.6.0 reduces boxing/unboxing overhead on some EVM opcodes, like PushX and Caller

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants