Page MenuHomePhabricator
Differential D13028

[SQLite] implement resetting Outbound P2P messages
ClosedPublic
Actions

Authored by kamil on Aug 8 2024, 4:23 AM.
Tags
None
Referenced Files
F2755052: D13028.diff
Wed, Sep 18, 9:25 PM
Unknown Object (File)
Sat, Sep 7, 1:47 PM
Unknown Object (File)
Sat, Sep 7, 11:37 AM
Unknown Object (File)
Sat, Sep 7, 11:36 AM
Unknown Object (File)
Wed, Sep 4, 2:24 AM
Unknown Object (File)
Sat, Aug 31, 11:18 AM
Unknown Object (File)
Fri, Aug 30, 2:10 PM
Unknown Object (File)
Thu, Aug 29, 3:53 AM
View All 24 Files
Subscribers
ashoat

Details

Reviewers
tomek
Commits
rCOMMdacfe284e5b0: [SQLite] implement resetting Outbound P2P messages
Summary

ENG-6982.

It is possible to do it as one query but will be more complicated and less readable, I can update if reviewers think it worth it.

Each query is described in code comments.

Test Plan

Tests

Diff Detail

Repository
rCOMM Comm
Branch
resending-2
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Aug 8 2024, 4:23 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptAug 8 2024, 4:23 AM
kamil added a child revision: D13029: [web] introduce resetting Outbound P2P Messages to shared worker.Aug 8 2024, 4:25 AM
Harbormaster failed remote builds in B31007: Diff 43256!Aug 8 2024, 4:46 AM
kamil published this revision for review.Aug 8 2024, 5:06 AM
tomek accepted this revision.Aug 12 2024, 1:11 AM
tomek added inline comments.
native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp
2580–2611

What happens if one of them succeeds and one fails?

This revision is now accepted and ready to land.Aug 12 2024, 1:11 AM
kamil added inline comments.Aug 13 2024, 3:26 AM
native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp
2580–2611

good call, I am going to wrap this in transaction in the following diffs

Harbormaster completed remote builds in B31007: Diff 43256.Aug 13 2024, 3:43 AM
Closed by commit rCOMMdacfe284e5b0: [SQLite] implement resetting Outbound P2P messages (authored by kamil). · Explain WhyAug 13 2024, 4:10 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMdacfe284e5b0: [SQLite] implement resetting Outbound P2P messages.

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
DatabaseManagers/
2 lines
2 lines
78 lines
web/
cpp/
3 lines
shared-worker/
_generated/
queries/
82 lines
types/
1 line

Diff 43256

View Options

native/cpp/CommonCpp/DatabaseManagers/DatabaseQueryExecutor.h

Loading...
View Options

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.h

Loading...
View Options

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp

Loading...
View Options

web/cpp/SQLiteQueryExecutorBindings.cpp

Loading...
View Options

web/shared-worker/_generated/comm_query_executor.wasm

Loading...
View Options

web/shared-worker/queries/outbound-p2p-message-queries.test.js

Loading...
View Options

web/shared-worker/types/sqlite-query-executor.js

Loading...