Page MenuHomePhabricator
Differential D11923

[native] Delete unused functions
ClosedPublic
Actions

Authored by tomek on Tue, May 7, 8:43 AM.
Tags
None
Referenced Files
F1786193: D11923.id39897.diff
Sat, May 18, 3:57 PM
F1785071: D11923.id39896.diff
Sat, May 18, 12:54 PM
Unknown Object (File)
Sat, May 18, 2:52 AM
Unknown Object (File)
Fri, May 17, 11:03 AM
Unknown Object (File)
Thu, May 16, 2:49 PM
Unknown Object (File)
Mon, May 13, 9:07 PM
Unknown Object (File)
Sat, May 11, 11:35 PM
Unknown Object (File)
Sat, May 11, 3:39 PM
View All 11 Files
Subscribers
ashoat

Details

Reviewers
kamil
marcin
varun
Commits
rCOMM94b3621f714b: [native] Delete unused functions
Summary

All the async functions that process a single store can now be deleted - we should only use the new function which processes all the stores in a single transaction.

https://linear.app/comm/issue/ENG-6004/wrap-all-store-operations-from-one-action-in-transaction

Depends on D11922

Test Plan

Checked if persisting drafts still works.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek created this revision.Tue, May 7, 8:43 AM
Herald added a subscriber: ashoat. · View Herald TranscriptTue, May 7, 8:43 AM
Harbormaster completed remote builds in B28738: Diff 39896.Tue, May 7, 9:00 AM
tomek requested review of this revision.Tue, May 7, 9:00 AM
tomek added a child revision: D11931: [lib] Expose function that processes operations as a part of SQLite API.Wed, May 8, 3:16 AM
marcin accepted this revision.Wed, May 8, 3:19 AM

Checked if persisting drafts still works.

Same suggestions to test plan as for parent diff.

This revision is now accepted and ready to land.Wed, May 8, 3:19 AM
tomek added a comment.Thu, May 9, 4:51 AM
In D11923#341468, @marcin wrote:

Checked if persisting drafts still works.

Same suggestions to test plan as for parent diff.

Tested also messages and threads.

Closed by commit rCOMM94b3621f714b: [native] Delete unused functions (authored by tomek). · Explain WhyThu, May 9, 5:25 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMM94b3621f714b: [native] Delete unused functions.

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
NativeModules/
31 lines
68 lines
PersistentStorageUtilities/
DataStores/
50 lines
_generated/
schema/
41 lines

Diff 39992

View Options

native/cpp/CommonCpp/NativeModules/CommCoreModule.h

Loading...
View Options

native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp

Loading...
View Options

native/cpp/CommonCpp/NativeModules/PersistentStorageUtilities/DataStores/BaseDataStore.h

Loading...
View Options

native/cpp/CommonCpp/_generated/commJSI-generated.cpp

Loading...
View Options

native/cpp/CommonCpp/_generated/commJSI.h

Loading...
View Options

native/schema/CommCoreModuleSchema.js

Loading...