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

Authored by tomek on May 21 2024, 3:56 PM.

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

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.May 21 2024, 3:56 PM

Herald added a subscriber: ashoat. · View Herald TranscriptMay 21 2024, 3:56 PM

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

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, 3:59 PM

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

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, 7:23 PM

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

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

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

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

Rebase

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

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

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

Harbormaster failed remote builds in B29140: Diff 40508!

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

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

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

marcin added inline comments.

native/redux/persist.js
1310	Is this strictly necessary? Not opposed - just curious.

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

tomek removed a child revision: D12153: [native] Persist only lastUserInteractionCalendar from the entry store.May 22 2024, 10: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, 11:12 AM

native/redux/persist.js
1310	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, 11:16 AM

Closed by commit rCOMM3caaff0d3723: [native] Add a migration that moves entries to SQLite. · Explain WhyMay 27 2024, 12:50 PM

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 40508

View Options

[native] Add a migration that moves entries to SQLiteClosedPublicActions