Skip to content

refactor(innertx): init with reth tables macro #69

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.

Sign up for GitHub

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

Jump to bottom
Merged
louisliu2048 merged 5 commits into from
Jan 5, 2026
Merged

refactor(innertx): init with reth tables macro #69

louisliu2048 merged 5 commits into from
Jan 5, 2026

Conversation

@Vui-Chee
Copy link
Contributor

@Vui-Chee Vui-Chee commented Dec 30, 2025

Description

Update innertx to use canonical DB APIs from upstream reth to reduce boilerplate code and follow convention.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Other (please describe):

@Vui-Chee Vui-Chee changed the title refactor(innertx): refactor(innertx): reth tables macro to reduce boilerplate code Dec 30, 2025
@Vui-Chee
Merge branch 'main' into vui-chee/innertx-refactor-db
c175942 
* main:
  chore: fix e2e fb benchmark tests (#78)
  feat: update eth_subscribe behavior to allow no address filtering and sends one tx per message (#76)
  feat: add fb benchmark tests (#75)
  feat: advertise correct version (#74)
  improvement: widen check scope (#73)
  fix chainspec crate compile (#72)
@Vui-Chee
Merge branch 'main' into vui-chee/innertx-refactor-db
3f508e4 
* main:
  revert innerTx flashblock support PR (#77)
@Vui-Chee
properly init custom tables defined in macro
798fd46 
`tables!` auto-generates a `Tables` enum.
@Vui-Chee Vui-Chee changed the title refactor(innertx): reth tables macro to reduce boilerplate code refactor(innertx): init with reth tables macro Jan 5, 2026
JimmyShi22
JimmyShi22 approved these changes Jan 5, 2026
View reviewed changes
Copy link
Contributor

@JimmyShi22 JimmyShi22 left a comment

Choose a reason for hiding this comment

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

LGTM, use existing marco can simply codes

@louisliu2048 louisliu2048 merged commit ecafb35 into main Jan 5, 2026
@Vui-Chee Vui-Chee deleted the vui-chee/innertx-refactor-db branch January 5, 2026 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JimmyShi22 JimmyShi22 JimmyShi22 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Vui-Chee @JimmyShi22 @louisliu2048