[native] Add a migration that moves entries to SQLite
ClosedPublic
Actions

Authored by tomek on May 21 2024, 8:56 AM.

Details

Reviewers

kamil
marcin

Commits

rCOMM3caaff0d3723: [native] Add a migration that moves entries to SQLite

Summary

This migration saves the entries in SQLite

Depends on D12152

https://linear.app/comm/issue/ENG-3486/move-entrystore-to-sqlite

Test Plan

Killed a keyserver, opened the app - checked the logs, and verified that the migration was run. If the migration hasn't moved the entries successfully, there would be an alert reporting store inconsistencies.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.May 21 2024, 8:56 AM

Herald added a subscriber: ashoat. · View Herald TranscriptMay 21 2024, 8:56 AM

tomek added a child revision: D12153: [native] Persist only lastUserInteractionCalendar from the entry store.May 21 2024, 8:57 AM

tomek added a parent revision: D12152: [lib] Schedule entry store ops processing.

tomek removed a parent revision: D12156: [lib] Update delete keyserver, delete thread, leave thread and set new session actions with the ops approach.May 21 2024, 8:59 AM

Harbormaster returned this revision to the author for changes because remote builds failed.May 21 2024, 9:41 AM

Harbormaster failed remote builds in B29122: Diff 40489!

tomek added a child revision: D12162: [lib] Replace merging entries with an ops approach.May 21 2024, 12:23 PM

tomek removed a child revision: D12162: [lib] Replace merging entries with an ops approach.May 21 2024, 12:23 PM

tomek added a child revision: D12162: [lib] Replace merging entries with an ops approach.May 21 2024, 12:28 PM

tomek removed a child revision: D12162: [lib] Replace merging entries with an ops approach.May 21 2024, 12:33 PM

tomek edited the summary of this revision. (Show Details)May 21 2024, 12:42 PM

Rebase

tomek added a child revision: D12162: [lib] Replace merging entries with an ops approach.May 21 2024, 12:44 PM

tomek removed a child revision: D12162: [lib] Replace merging entries with an ops approach.May 21 2024, 12:45 PM

Harbormaster returned this revision to the author for changes because remote builds failed.May 21 2024, 12:57 PM

Harbormaster failed remote builds in B29140: Diff 40508!

Harbormaster completed remote builds in B29140: Diff 40508.May 21 2024, 1:15 PM

tomek requested review of this revision.May 21 2024, 1:15 PM

marcin accepted this revision.May 22 2024, 2:53 AM

marcin added inline comments.

native/redux/persist.js
1310 ↗	(On Diff #40508)	Is this strictly necessary? Not opposed - just curious.

This revision is now accepted and ready to land.May 22 2024, 2:53 AM

tomek removed a child revision: D12153: [native] Persist only lastUserInteractionCalendar from the entry store.May 22 2024, 3:48 AM

tomek added a child revision: D12156: [lib] Update delete keyserver, delete thread, leave thread and set new session actions with the ops approach.

tomek added inline comments.May 22 2024, 4:12 AM

native/redux/persist.js
1310 ↗	(On Diff #40508)	Maybe not necessary, but we're using the same approach in other migrations and I think it is a bit safer.

tomek edited the test plan for this revision. (Show Details)May 22 2024, 4:16 AM

Closed by commit rCOMM3caaff0d3723: [native] Add a migration that moves entries to SQLite (authored by tomek). · Explain WhyMay 27 2024, 5:50 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM3caaff0d3723: [native] Add a migration that moves entries to SQLite.

Revision Contents
Changeset List

Path

Size

native/

redux/

persist.js

36 lines

Diff 40664

View Options

[native] Add a migration that moves entries to SQLiteClosedPublicActions