Skip to content

Migrate company caching to ID-based deduplicated keyspace - #53

Merged
dontlaugh merged 4 commits into
mainfrom
caching-updates
Feb 24, 2026
Merged

dontlaugh merged 4 commits into
mainfrom
caching-updates

Conversation

@dontlaugh

Copy link
Copy Markdown
Contributor

Instead of storing the full company JSON at every lookup key, lookup
keys now store just the company ID string and a single ID-based
primary key stores the full company. The full company data structure
is only stored in one place.

@dontlaugh
dontlaugh marked this pull request as ready for review February 12, 2026 16:18
@dontlaugh
dontlaugh requested a review from a team as a code owner February 12, 2026 16:18

@bpapillon bpapillon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Plan here is to get clients (Schematic/Go and Airia/C#) updated before they update to this version of the replicator?

Comment thread cache.go
Instead of storing the full company JSON at every lookup key, lookup
keys now store just the company ID string and a single ID-based
primary key stores the full company. The full company data structure
is only stored in one place.
schematic:company:{version}:{companyID}
Comment thread cache.go Outdated
Pass p.pattern instead of constructing a new pattern inline.
@dontlaugh
dontlaugh requested a review from cbrady February 19, 2026 16:20
@dontlaugh
dontlaugh merged commit a235954 into main Feb 24, 2026
2 checks passed
@dontlaugh
dontlaugh deleted the caching-updates branch February 24, 2026 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants