Page MenuHomePhabricator

[SQLite] implement resetting Outbound P2P messages
ClosedPublic

Authored by kamil on Aug 8 2024, 4:23 AM.
Tags
None
Referenced Files
F3389414: D13028.id43328.diff
Fri, Nov 29, 7:00 PM
F3388611: D13028.diff
Fri, Nov 29, 3:09 PM
Unknown Object (File)
Wed, Nov 20, 3:10 AM
Unknown Object (File)
Wed, Nov 20, 3:09 AM
Unknown Object (File)
Wed, Nov 20, 2:44 AM
Unknown Object (File)
Sat, Nov 9, 10:12 AM
Unknown Object (File)
Oct 14 2024, 7:19 PM
Unknown Object (File)
Oct 14 2024, 7:19 PM
Subscribers

Details

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 held this revision as a draft.
kamil published this revision for review.Aug 8 2024, 5:06 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
native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp
2580–2611

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