Skip to content

Comments

feat: add transaction index to batch execution error handling#3000

Merged
rakita merged 3 commits intobluealloy:mainfrom
crStiv:todo
Sep 25, 2025
Merged

feat: add transaction index to batch execution error handling#3000
rakita merged 3 commits intobluealloy:mainfrom
crStiv:todo

Conversation

@crStiv
Copy link
Contributor

@crStiv crStiv commented Sep 19, 2025

Add TransactionIndexedError type to include transaction index when batch transaction execution fails.

Changes:

  • Add TransactionIndexedError<Error> struct in context/interface/src/result.rs
  • Update transact_many, transact_many_finalize, and transact_many_commit methods to return indexed errors
  • Add comprehensive tests for the new functionality
  • Remove TODO comment about adding tx index to error

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 19, 2025

CodSpeed Performance Report

Merging #3000 will not alter performance

Comparing crStiv:todo (af8ac8a) with main (e3d300c)

Summary

✅ 173 untouched

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

Makes sense and change lgtm. Will include it after next release

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

lgtm

@rakita rakita merged commit 0e05a30 into bluealloy:main Sep 25, 2025
31 checks passed
@github-actions github-actions bot mentioned this pull request Oct 5, 2025
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.

2 participants