Page MenuHomePhabricator

Read materialized project members instead of real members
ClosedPublic

Authored by epriestley on Dec 27 2015, 12:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 22, 7:06 PM
Unknown Object (File)
Sun, Mar 10, 7:13 AM
Unknown Object (File)
Tue, Mar 5, 3:46 PM
Unknown Object (File)
Tue, Mar 5, 3:46 PM
Unknown Object (File)
Tue, Mar 5, 3:46 PM
Unknown Object (File)
Tue, Mar 5, 3:46 PM
Unknown Object (File)
Feb 3 2024, 8:05 AM
Unknown Object (File)
Jan 29 2024, 9:13 PM
Subscribers
None

Details

Summary

Ref T10010. This will allow us to find superprojects with withMemberPHIDs(...) queries.

  • Copy all the current real member edges to materialized member edges.
  • Redirect all reads to look at materialized members.
  • This table is already kept in sync by earlier work with indexing.

Basically, flow is:

  • Writes (joining, leaving, adding/removing members) write to the real member edge type.
  • After a project's members change, they're copied to the materialized member edge type for that project and all of its superprojects.
  • Reads look at materialized members, so "Parent" sees the members of "Child" and "Grandchild" as its own members, but we still have the "real members" edge type to keep track of "natural" or "direct" members.
Test Plan
  • Ran migration.
  • Ran unit tests.
  • Saw the same projects as projects I was a member of.
  • Added some var_dump() stuff to verify the Owners changed.
  • Used grep to look for other readers of this edge type.
  • Made some project updates.

Diff Detail

Repository
rP Phabricator
Branch
sub6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 9813
Build 11809: Run Core Tests
Build 11808: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Read materialized project members instead of real members.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 27 2015, 3:07 PM
This revision was automatically updated to reflect the committed changes.