Page MenuHomePhabricator
Differential D11453

Revert "[native][web] Read integrity thread hashes from SQLite on app start"
AbandonedPublic
Actions

Authored by will on Mar 28 2024, 11:08 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 6, 7:35 AM
Unknown Object (File)
Thu, Oct 17, 11:27 AM
Unknown Object (File)
Thu, Oct 17, 11:27 AM
Unknown Object (File)
Thu, Oct 17, 11:27 AM
Unknown Object (File)
Thu, Oct 17, 11:27 AM
Unknown Object (File)
Tue, Oct 15, 8:25 AM
Unknown Object (File)
Sep 12 2024, 11:10 PM
Unknown Object (File)
Sep 6 2024, 3:08 AM
View All 40 Files
Subscribers
tomek

Details

Reviewers
ashoat
ginsu
atul
Summary

This reverts commit 5695216e237f08eae63e3c845481ab791528b18e.

Depends on D11456

Test Plan

Reverts reading sqlite on app start to fix an urgent issue with compiling on native

Context: https://linear.app/comm/issue/ENG-7597/app-crashes-when-running-migration-65#comment-8b4375fe

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

will created this revision.Mar 28 2024, 11:08 AM
Herald added a subscriber: tomek. · View Herald TranscriptMar 28 2024, 11:08 AM
will edited the summary of this revision. (Show Details)Mar 28 2024, 11:10 AM
will added a parent revision: D11454: [sqlite] downgrade migration version on native from '65' to '64'.
Harbormaster returned this revision to the author for changes because remote builds failed.Mar 28 2024, 11:19 AM
Harbormaster failed remote builds in B27791: Diff 38486!
will updated this revision to Diff 38490.Mar 28 2024, 11:26 AM
will edited the summary of this revision. (Show Details)

rebase

will edited the summary of this revision. (Show Details)Mar 28 2024, 11:27 AM
will added a parent revision: D11456: Revert "[lib] integrity reducer tests".
Harbormaster completed remote builds in B27795: Diff 38490.Mar 28 2024, 11:42 AM
will requested review of this revision.Mar 28 2024, 11:42 AM
ashoat accepted this revision.Mar 28 2024, 11:46 AM
This revision is now accepted and ready to land.Mar 28 2024, 11:46 AM
will abandoned this revision.Mar 28 2024, 1:48 PM

https://linear.app/comm/issue/ENG-7597/app-crashes-when-running-migration-65#comment-8b4375fe resolved

Revision Contents
Changeset List

PathSize
lib/
reducers/
41 lines
1 line
1 line
types/
2 lines
native/
data/
7 lines
web/
shared-worker/
utils/
9 lines

Diff 38490

View Options

lib/reducers/integrity-reducer.js

Loading...
View Options

lib/reducers/master-reducer.js

Loading...
View Options

lib/reducers/message-reducer.test.js

Loading...
View Options

lib/types/store-ops-types.js

Loading...
View Options

native/data/sqlite-data-handler.js

Loading...
View Options

web/shared-worker/utils/store.js

Loading...