[SQLite] remove old architecture comments
ClosedPublic
Actions

Authored by kamil on May 20 2025, 11:28 AM.

Details

Reviewers

bartek
tomek
angelika

Commits

rCOMMc125e742c140: [SQLite] remove old architecture comments

Summary

Implemented as part of ENG-10603

Deprecating the old design, we want multiple at once, not to decide on preprocessing time

Depends on D14718

Test Plan

N/A

Diff Detail

Repository

rCOMM Comm

Branch

publish

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.May 20 2025, 11:28 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptMay 20 2025, 11:28 AM

Harbormaster completed remote builds in B34201: Diff 48132.May 20 2025, 12:11 PM

kamil added a child revision: D14720: [SQLite] temporary make `SQLiteQueryExecutor` methods and fields public.May 20 2025, 1:58 PM

kamil published this revision for review.May 20 2025, 3:58 PM

bartek accepted this revision.May 22 2025, 7:37 AM

This revision is now accepted and ready to land.May 22 2025, 7:37 AM

Closed by commit rCOMMc125e742c140: [SQLite] remove old architecture comments. · Explain WhyMay 26 2025, 12:11 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMc125e742c140: [SQLite] remove old architecture comments.

Revision Contents
Changeset List

Path

Size

native/

cpp/

CommonCpp/

DatabaseManagers/

DatabaseManager.h

1 line

DatabaseManager.cpp

2 lines

Diff 48132

View Options

native/cpp/CommonCpp/DatabaseManagers/DatabaseManager.h