Page MenuHomePhabricator
Differential D7937

[native] start processing report store operations on database
ClosedPublic
Actions

Authored by kamil on May 23 2023, 3:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 16, 2:54 AM
Unknown Object (File)
Mon, Apr 15, 3:56 PM
Unknown Object (File)
Sun, Apr 14, 7:58 AM
Unknown Object (File)
Sat, Apr 13, 3:55 PM
Unknown Object (File)
Sat, Apr 13, 12:20 PM
Unknown Object (File)
Fri, Apr 12, 4:24 AM
Unknown Object (File)
Thu, Apr 11, 9:10 AM
Unknown Object (File)
Tue, Apr 9, 10:48 PM
View All 44 Files
Subscribers
ashoat
atul

Details

Reviewers
tomek
michal
Commits
rCOMM3294c18298d0: [native] start processing report store operations on database
Summary

Enable running ops on SQLite.

Depends on D7936, D7933

Test Plan

Create some ops in reducer and check if they're properly processed on SQLite

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.May 23 2023, 3:28 AM
kamil held this revision as a draft.
Herald added subscribers: atul, ashoat. · View Herald TranscriptMay 23 2023, 3:28 AM
kamil added a child revision: D7938: [native] fetch reports from SQLite and dispatch to store.May 23 2023, 3:44 AM
Harbormaster completed remote builds in B19649: Diff 26857.May 23 2023, 3:46 AM
kamil published this revision for review.May 23 2023, 4:55 PM
tomek accepted this revision.May 24 2023, 3:11 AM
tomek added inline comments.
lib/ops/report-store-ops.js
90 ↗(On Diff #26857)

Should we e.g. throw when an unsupported operation appears?

This revision is now accepted and ready to land.May 24 2023, 3:11 AM
kamil edited parent revisions, added: D7977: [native] migrate reports to SQLite; removed: D7936: [CommCoreModule] implement getting report store from database.May 25 2023, 7:57 AM
kamil updated this revision to Diff 27297.May 31 2023, 8:28 AM
kamil added a parent revision: D7936: [CommCoreModule] implement getting report store from database.

rebase before landing

lib/ops/report-store-ops.js
90 ↗(On Diff #26857)

In other cases we throw from C++ level, I'll follow current approach

kamil removed a parent revision: D7936: [CommCoreModule] implement getting report store from database.May 31 2023, 8:29 AM
Harbormaster completed remote builds in B19922: Diff 27297.May 31 2023, 8:48 AM
Closed by commit rCOMM3294c18298d0: [native] start processing report store operations on database (authored by kamil). · Explain WhyMay 31 2023, 9:24 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM3294c18298d0: [native] start processing report store operations on database.

Revision Contents
Changeset List

PathSize
native/
redux/
3 lines
13 lines

Diff 27304

View Options

native/redux/redux-setup.js

Loading...
View Options

native/redux/redux-utils.js

Loading...