Skip to content

Richer progress report#426

Open
driv3r wants to merge 7 commits intomainfrom
feature/meaningful-progress-report
Open

Richer progress report#426
driv3r wants to merge 7 commits intomainfrom
feature/meaningful-progress-report

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
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.

1 participant