Differential D8384
[native] Add utils for migrating message store Authored by michal on Jun 30 2023, 3:15 AM. Tags None Referenced Files
Subscribers
Details
https://linear.app/comm/issue/ENG-3995/create-functions-for-migrating-the-client-stores Depends on D8355 Tested using the id schema migration introduced in the next diff in stack. Checked if the migration run correctly
Diff Detail
Event TimelineComment Actions Similar feedback to D8385. I think you should update functions to as input takes data from the database (fetched only once), return only operations, and later process those at once. Also, I do not feel comfortable about mixing process*StoreOperationsSync and process*StoreOperations in one migration so probably you can stick to only one option and after generating all operations for migrating data do something similar to processDBStoreOperations.
Comment Actions Update the migration functions so instead of fetching the stores, they take them as an argument and return operations. | ||||||||||||||||||||||||||||||||||||||