Skip to content

Comments

[MongoDB Storage] Revert logic for bucket checksum pre-calculations#511

Merged
rkistner merged 4 commits intomainfrom
fix-checksum-precalculation
Feb 18, 2026
Merged

[MongoDB Storage] Revert logic for bucket checksum pre-calculations#511
rkistner merged 4 commits intomainfrom
fix-checksum-precalculation

Conversation

@rkistner
Copy link
Contributor

#503 made some changes to optimize the compact process.

However, the changes were also used for bucket checksum pre-calculations, where the same logic does not work:

  1. Checksum pre-calculations specifically need to be resumable after a process restart, while a compact process doesn't have to be.
  2. The logic for pre-calcuations sometimes skipped buckets to process in the next batch, but the updated logic would not return the same buckets again.
  3. No concurrent changes are performed during checksum pre-calculations, so buckets changing during processing is not relevant there.

This reverts checksum pre-calculations to work like it did before - we now use different queries for compacting versus checksum pre-calculations.

@changeset-bot
Copy link

changeset-bot bot commented Feb 17, 2026

🦋 Changeset detected

Latest commit: 4fa98d2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 12 packages
Name Type
@powersync/service-module-mongodb-storage Patch
@powersync/service-schema Patch
@powersync/service-module-mongodb Patch
@powersync/service-module-mssql Patch
@powersync/service-module-mysql Patch
@powersync/service-module-postgres Patch
@powersync/service-image Patch
@powersync/service-core Patch
@powersync/service-core-tests Patch
@powersync/service-module-core Patch
@powersync/service-module-postgres-storage Patch
test-client Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@rkistner rkistner merged commit e21db9b into main Feb 18, 2026
28 checks passed
@rkistner rkistner deleted the fix-checksum-precalculation branch February 18, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants