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

fix: event-replay block parsing outdated and incorrect #2133

Merged
merged 4 commits into from
Oct 21, 2024

Conversation

zone117x
Copy link
Member

The event-replay logic was using a copy&pasted function for parsing /new_block data, which had become outdated and incorrect.

This removes the duplicated code, and both event-replay and regular block ingestion use the same (correct) block parsing code.

Copy link

codecov bot commented Oct 21, 2024

Codecov Report

Attention: Patch coverage is 96.96970% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...play/parquet-based/importers/new-block-importer.ts 0.00% 1 Missing ⚠️
src/event-stream/event-server.ts 98.46% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link

github-actions bot commented Oct 21, 2024

Vercel deployment URL: https://stacks-blockchain-hvppvv6qd-hirosystems.vercel.app 🚀

Copy link
Collaborator

@rafaelcr rafaelcr left a comment

Choose a reason for hiding this comment

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

💯

@zone117x zone117x merged commit 2cd69fa into develop Oct 21, 2024
27 checks passed
@zone117x zone117x deleted the fix/event-replay-bugs branch October 21, 2024 14:20
blockstack-devops pushed a commit that referenced this pull request Oct 23, 2024
## [8.2.0-beta.1](v8.1.2...v8.2.0-beta.1) (2024-10-23)

### Features

* allow stackerdb_chunks messages to be stored in db raw events table ([d03f2ef](d03f2ef))
* include `tenure-height` in block responses ([#2134](#2134)) ([07426a2](07426a2))
* ingest `signer_signature` from `/new_block` event and expose in new endpoint ([#2125](#2125)) ([c389154](c389154))

### Bug Fixes

* event-replay block parsing outdated and incorrect ([#2133](#2133)) ([2cd69fa](2cd69fa))
@blockstack-devops
Copy link

🎉 This PR is included in version 8.2.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

blockstack-devops pushed a commit that referenced this pull request Oct 25, 2024
## [8.2.0](v8.1.2...v8.2.0) (2024-10-25)

### Features

* allow stackerdb_chunks messages to be stored in db raw events table ([d03f2ef](d03f2ef))
* include `tenure-height` in block responses ([#2134](#2134)) ([07426a2](07426a2))
* ingest `signer_signature` from `/new_block` event and expose in new endpoint ([#2125](#2125)) ([c389154](c389154))

### Bug Fixes

* event-replay block parsing outdated and incorrect ([#2133](#2133)) ([2cd69fa](2cd69fa))
@blockstack-devops
Copy link

🎉 This PR is included in version 8.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants