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.

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

Fix parquet default processor #458

Merged
merged 2 commits into from
Jul 15, 2024
Merged

Fix parquet default processor #458

merged 2 commits into from
Jul 15, 2024

Conversation

yuunlimm
Copy link
Contributor

@yuunlimm yuunlimm commented Jul 12, 2024 •
edited
Loading

Description

  • The rootcause of the failure is from wrong mapping between write_set_changes and WriteSetOpSizeInfo, where write_set_size_info is per state_key_hash not per changes
  • synced with the team offline, and we are aligned on having a separate table to close this tranche 2.

Test Plan

deployed on testnet
Screenshot 2024-07-15 at 9 44 18 AM

@yuunlimm yuunlimm marked this pull request as draft July 12, 2024 06:23
@yuunlimm yuunlimm force-pushed the yuunlimm/empty-migration-query branch 3 times, most recently from c632e79 to c5e900d Compare July 12, 2024 16:19
@yuunlimm yuunlimm force-pushed the yuunlimm/empty-migration-query branch from e48b769 to 84e2a55 Compare July 13, 2024 01:19
@yuunlimm yuunlimm changed the title fix diesel migration failure Fix parquet default processor Jul 13, 2024
@yuunlimm yuunlimm marked this pull request as ready for review July 15, 2024 16:43
@yuunlimm yuunlimm requested a review from a team July 15, 2024 16:44
@yuunlimm yuunlimm merged commit fbffc33 into main Jul 15, 2024
7 checks passed
@yuunlimm yuunlimm deleted the yuunlimm/empty-migration-query branch July 15, 2024 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rtso rtso rtso 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.
2 participants
@yuunlimm @rtso