Page MenuHomePhabricator
Differential D11576

[native/sqlite] Create thread activity table in sqlite
ClosedPublic
Actions

Authored by will on Mon, Apr 8, 6:18 AM.

Details

Reviewers
kamil
atul
ginsu
Commits
rCOMMedad6555237f: [native/sqlite] Create thread activity table in sqlite
Summary

Initializes the sqlite table for thread activity

Depends on D11575

Test Plan

Verified on database explorer the table was being created

Tested create_schema by building on new target

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

will created this revision.Mon, Apr 8, 6:18 AM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptMon, Apr 8, 6:18 AM
will edited the summary of this revision. (Show Details)Mon, Apr 8, 6:18 AM
will added a parent revision: D11575: [lib] Add thread activity store ops.
will added a child revision: D11578: [lib] refactor thread activity reducer to use sqlite operations.Mon, Apr 8, 6:25 AM
Harbormaster completed remote builds in B28046: Diff 38896.Mon, Apr 8, 6:33 AM
will requested review of this revision.Mon, Apr 8, 6:33 AM
atul accepted this revision.Mon, Apr 8, 10:27 AM

Would be good to include both create_schema and migration flow in Test Plan

This revision is now accepted and ready to land.Mon, Apr 8, 10:27 AM
will added a comment.Mon, Apr 8, 10:32 AM
In D11576#333165, @atul wrote:

Would be good to include both create_schema and migration flow in Test Plan

Cool. I'll add create_schema to this test plan. The migration flow I'll test in later diffs

will edited the test plan for this revision. (Show Details)Mon, Apr 15, 5:14 AM
Closed by commit rCOMMedad6555237f: [native/sqlite] Create thread activity table in sqlite (authored by will). · Explain WhyFri, Apr 19, 12:32 PM
This revision was automatically updated to reflect the committed changes.
will added a commit: rCOMMedad6555237f: [native/sqlite] Create thread activity table in sqlite.

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
DatabaseManagers/
17 lines
web/
shared-worker/
_generated/

Diff 39313

View Options

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp

Loading...
View Options

web/shared-worker/_generated/comm_query_executor.wasm

Loading...