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: skip empty chunks when decoding gz #456

Merged
merged 6 commits into from
Oct 23, 2023
Merged

Conversation

MicaiahReid
Copy link
Contributor

No description provided.

@MicaiahReid MicaiahReid temporarily deployed to Development-mainnet October 18, 2023 21:28 — with GitHub Actions Inactive
@lgalabru
Copy link
Contributor

I am taking over this PR, thank you for the progress made on this issue @MicaiahReid 🙏

@lgalabru lgalabru self-assigned this Oct 23, 2023
@lgalabru lgalabru temporarily deployed to Development-mainnet October 23, 2023 17:43 — with GitHub Actions Inactive
@lgalabru lgalabru temporarily deployed to Development-testnet October 23, 2023 17:43 — with GitHub Actions Inactive
@lgalabru
Copy link
Contributor

I've run an ingestion locally successfully, and just deployed the latest image to dev/testnet -> dev/mainnet -> stg/testnet -> stg/mainnet - and all the ingestions went well. Not sure what went wrong with last week deployment, but it looks
I'll do some additional checks, but this PR is looking stable to me - will be merging soon.

@lgalabru lgalabru had a problem deploying to Development-mainnet October 23, 2023 18:07 — with GitHub Actions Failure
@lgalabru lgalabru had a problem deploying to Development-testnet October 23, 2023 18:07 — with GitHub Actions Failure
@lgalabru lgalabru merged commit d729c7a into develop Oct 23, 2023
3 of 5 checks passed
@lgalabru lgalabru deleted the fix-tsv-ingestion-again branch October 23, 2023 20:09
Copy link

🎉 This PR is included in version 1.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants