Skip to content

Opt snpshotkv use journal entry - #1

Closed
songgaoye wants to merge 2 commits into
mainfrom
opt_snpshotkv_use_journalEntry
Closed

Opt snpshotkv use journal entry#1
songgaoye wants to merge 2 commits into
mainfrom
opt_snpshotkv_use_journalEntry

Conversation

@songgaoye

Copy link
Copy Markdown
Owner

Description

Closes: #XXXX


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • tackled an existing issue or discussed with a team member
  • left instructions on how to review the changes
  • targeted the main branch

Reviewers Checklist

All items are required.
Please add a note if the item is not applicable
and please add your handle next to the items reviewed
if you only reviewed selected items.

I have...

  • added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • confirmed all author checklist items have been addressed
  • confirmed that this PR does not change production code
  • reviewed content
  • tested instructions (if applicable)
  • confirmed all CI checks have passed

@github-actions

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days-before-close if no further activity occurs.

@github-actions github-actions Bot closed this Sep 19, 2025
songgaoye pushed a commit that referenced this pull request May 27, 2026
cosmos#1132)

* fix: evm tx index and log indexes are not patched

* patch

* cleanup ResetTransientGasUsed

* add fallback

* align patch
* cleanup

* more test

* add doc

* only overwrite tx index not found

* cleanup

* bump sdk

* fix lint later

* fix resolve

* cleanup

* Merge pull request #1 from canu0205/chris/fix-patch

fix: patch expected failure txs

* fix build

* lint

* cleanup

* continue

* show fail tx

* revert

* return err

* alt design: keep ctx.TxIndex plumbing, narrow PatchTxResponses to log indices

* add stm tx index tests

* changelog fix

* fix: ethTxIndex counter diverges from the ante-handler-emitted AttributeKeyTxIndex in mixed blocks

* fix: Hard failure on unmarshal for any successful non-EVM tx

* readd txindex correction

* comment

* lints

* add edge case tests

* fix double-counting

* fix divergence on ante failure

* convert FindEthTxIndexByHash to method

---------

Co-authored-by: mmsqe <tqd0800210105@gmail.com>
Co-authored-by: Alex | Cosmos Labs <alex@cosmoslabs.io>
Co-authored-by: chris <chancold123@gmail.com>
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.

1 participant