Page MenuHomePhabricator

[sqlite] introduce aux user store C++ methods
ClosedPublic

Authored by will on Mar 31 2024, 10:16 PM.
Tags
None
Referenced Files
F3380464: D11508.diff
Wed, Nov 27, 11:57 PM
Unknown Object (File)
Sun, Nov 3, 1:50 AM
Unknown Object (File)
Oct 14 2024, 3:35 PM
Unknown Object (File)
Oct 14 2024, 3:35 PM
Unknown Object (File)
Oct 14 2024, 3:35 PM
Unknown Object (File)
Sep 17 2024, 6:34 PM
Unknown Object (File)
Sep 4 2024, 1:11 AM
Unknown Object (File)
Sep 4 2024, 1:11 AM
Subscribers

Details

Summary

This implements the C++ SQLite operations for the aux user store

Depends on D11507

Test Plan

ran aux user queries test successfully

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

will requested review of this revision.Mar 31 2024, 10:30 PM

This looks correct, but would be great to have more thorough Test Plan. The aux-user-queries.tests are great though!

native/cpp/CommonCpp/DatabaseManagers/DatabaseQueryExecutor.h
90–94 ↗(On Diff #38621)

Assuming this was all codegen'd so not reviewing

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.h
103–106 ↗(On Diff #38621)

Assuming this was codegen'd so not reviewing

native/cpp/CommonCpp/DatabaseManagers/entities/AuxUserInfo.h
23–24 ↗(On Diff #38621)

Let's add newline at end of file

This revision is now accepted and ready to land.Apr 1 2024, 1:44 PM
native/cpp/CommonCpp/DatabaseManagers/DatabaseQueryExecutor.h
90–94 ↗(On Diff #38621)

These are actually not code generated. But tested these operations pretty thoroughly later in the stack so should be fine.

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.h
103–106 ↗(On Diff #38621)

Same comment as above. This isn't code generated but tested later in the stack.