Details

Reviewers

atul
inka
kamil

Commits

rCOMM5d3f1a74764f: [sqlite] add communites table for community store data

Summary

Introduce migrations + modified initial create_schema sql query to introduce communities table to store community store data in SQLite

Depends on D11070

Test Plan

Test migration with new user and preexisting user and was able to see communities table

Screenshot 2024-02-15 at 5.04.59 PM.png (2×3 px, 994 KB)

Web logs showing we are on db version 35:

Screenshot 2024-02-16 at 3.39.19 PM.png (1×1 px, 444 KB)

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ginsu created this revision.Feb 15 2024, 2:55 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptFeb 15 2024, 2:55 AM

ginsu retitled this revision from [sqlite] add communites table for community store data to DRAFT [sqlite] add communites table for community store data.Feb 15 2024, 2:57 AM

ginsu edited the summary of this revision. (Show Details)

ginsu edited the test plan for this revision. (Show Details)

ginsu added reviewers: atul, inka, kamil.

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 15 2024, 3:06 AM

Harbormaster failed remote builds in B26847: Diff 37111!

update

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 15 2024, 3:24 AM

Harbormaster failed remote builds in B26848: Diff 37112!

rebase

ginsu retitled this revision from DRAFT [sqlite] add communites table for community store data to [sqlite] add communites table for community store data.Feb 15 2024, 10:16 PM

ginsu edited the summary of this revision. (Show Details)

ginsu edited the test plan for this revision. (Show Details)

ginsu added a parent revision: D11070: [lib] introduce CommunityStore spec.

Harbormaster completed remote builds in B26943: Diff 37285.Feb 15 2024, 10:28 PM

ginsu requested review of this revision.Feb 15 2024, 10:28 PM

Looks good, two nits:

The file from the test plan is not visible
Can you also check if it works on the web? Logs on the worker are enough - here is how you can access them

This revision is now accepted and ready to land.Feb 16 2024, 3:05 PM

ginsu attached a referenced file: F1164684: Screenshot 2024-02-15 at 5.04.59 PM.png. (Show Details)Feb 16 2024, 8:30 PM

In D11087#320698, @kamil wrote:

Looks good, two nits:

The file from the test plan is not visible

Can you also check if it works on the web? Logs on the worker are enough - here is how you can access them

Updated test plan!

@ginsu I see you attached .wasm file so it's probably okay but it would be safer if you could wait with landing until resolving ENG-6879.

fix merge conflict with .wasm file

Harbormaster completed remote builds in B27077: Diff 37458.Feb 22 2024, 10:21 PM

ginsu added a child revision: D11151: [sqlite] add methods to operate on communities table.Feb 23 2024, 8:00 AM

In D11087#321178, @kamil wrote:

@ginsu I see you attached .wasm file so it's probably okay but it would be safer if you could wait with landing until resolving ENG-6879.

Issue is fixed - you can rebase and should be fine to land

rebase before landing

This revision was landed with ongoing or failed builds.Feb 28 2024, 4:02 AM

Closed by commit rCOMM5d3f1a74764f: [sqlite] add communites table for community store data. · Explain Why

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM5d3f1a74764f: [sqlite] add communites table for community store data.

Harbormaster failed remote builds in B27213: Diff 37666!Feb 28 2024, 9:37 AM

[sqlite] add communites table for community store data
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 37673

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp

web/database/_generated/comm_query_executor.wasm

[sqlite] add communites table for community store dataClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 37673

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp

web/database/_generated/comm_query_executor.wasm

[sqlite] add communites table for community store data
ClosedPublic
Actions

Revision Contents
Changeset List