Skip to content

Conversation

@driv3r
Copy link
Contributor

@driv3r driv3r commented Feb 6, 2026

Added

  • PaginationKey now includes column name

Changed

  • Use PaginationKey instead of raw uint64 for progress report. This means that table progress report will
    include not raw value, but a whole PaginationKey object, i.e.
    {
      "type": "uint64",
      "column": "id",
      "value": 999
    }
    which will be in line with the format of what we have in the state dump
  • Refactored MarshalPaginationKey function into MarshalJSON implementations, so there's less work during progress serialization

@driv3r driv3r marked this pull request as ready for review February 6, 2026 19:51
@driv3r driv3r self-assigned this Feb 6, 2026
@driv3r driv3r requested review from a team, grodowski and milanatshopify February 6, 2026 19:51
@driv3r driv3r merged commit 86bced2 into main Feb 12, 2026
45 of 48 checks passed
@driv3r driv3r deleted the feature/meaningful-progress-report branch February 12, 2026 14:55
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