Skip to content

MS-997 Reset local records when sync partitioning changes#1219

Merged
luhmirin-s merged 1 commit into
mainfrom
feature/MS-997-sync-partition-change-handling
Jul 1, 2025
Merged

MS-997 Reset local records when sync partitioning changes#1219
luhmirin-s merged 1 commit into
mainfrom
feature/MS-997-sync-partition-change-handling

Conversation

@luhmirin-s
Copy link
Copy Markdown
Contributor

@luhmirin-s luhmirin-s commented Jun 11, 2025

JIRA ticket
Will be released in: version

Root cause analysis (for bugfixes only)

First known affected version: 2025.1.0

  • When sync partitioning changes in the project configuration, already downloaded records remain in the local database, causing a mismatch in numbers.
  • Additional local records might also affect the matching speed and results by returning records that shouldn't have been synced (if the ID pool type does not match sync partitioning).

Notable changes

  • On the project configuration sync, if the change in sync partitioning has been detected, all local records are wiped, and a new sync is scheduled.
  • Change of sync partitioning is an extraordinary circumstance, and some disruption should be expected. Creating more complex diff handling is possible, but the ROI would not be justify it.

Testing guidance

  • Create/configure a project with project sync scope and some records.
  • Log into SID and sync the records.
  • Change sync scope to modules and re-add known module names in Vulcan.
  • Open the SID dashboard settings and sync configuration.
  • Go to Sync Info and select some modules to down sync.
  • Check that the number of records in synced modules matches "Total on device".

Testiny case

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@luhmirin-s luhmirin-s requested review from a team, BurningAXE, TristramN, alex-vt, alexandr-simprints, meladRaouf and ybourgery and removed request for a team June 11, 2025 08:26
@cla-bot cla-bot Bot added the ... label Jun 11, 2025
@luhmirin-s luhmirin-s force-pushed the feature/MS-997-sync-partition-change-handling branch from f3b019d to 704e128 Compare June 17, 2025 07:12
@luhmirin-s luhmirin-s force-pushed the feature/MS-997-sync-partition-change-handling branch from 704e128 to 646888d Compare June 26, 2025 07:33
@sonarqubecloud
Copy link
Copy Markdown

@luhmirin-s luhmirin-s merged commit c0b3432 into main Jul 1, 2025
20 of 21 checks passed
@luhmirin-s luhmirin-s deleted the feature/MS-997-sync-partition-change-handling branch July 1, 2025 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants