Apps/ImageApi
1
0 Fork 0
Code Issues 8 Pull Requests 1 Releases Wiki Activity

perf/faces-embeddings-no-clone #72

Merged
cameron merged 2 commits from perf/faces-embeddings-no-clone into master 2026-05-01 23:09:23 +00:00
Conversation 0 Commits 2 Files Changed 7
+86 -76
cameron commented 2026-05-01 23:09:15 +00:00
Owner
Copy link
Copy source
No description provided.
cameron added 2 commits 2026-05-01 23:09:15 +00:00
faces: avoid 40 MB row clone in /faces/embeddings 1d9b9a0bc4 
list_embeddings cloned the full FaceDetectionRow inside the filter_map
just to pair it with the base64-encoded embedding. The 2 KB BLOB was
already on the row — at 20k unassigned faces that's 40 MB of pointless
heap traffic per Apollo cluster-suggest run. Move the bytes out via
Option::take() so the row drops the BLOB instead of duplicating it.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
style: cargo fmt sweep fb4df4b195 
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
cameron merged commit 4340b164eb into master 2026-05-01 23:09:23 +00:00
cameron deleted branch perf/faces-embeddings-no-clone 2026-05-01 23:09:23 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
needs-info
Waiting on reporter for more information
 needs-triage
Maintainer needs to evaluate this issue
 ready-for-agent
Fully specified, ready for an AFK agent
 ready-for-human
Requires human implementation
 wayfinder:grilling
Wayfinder child ticket: grilling
 wayfinder:map
Wayfinder map: the single issue holding Notes / Decisions-so-far / Fog
 wayfinder:prototype
Wayfinder child ticket: prototype
 wayfinder:research
Wayfinder child ticket: research
 wayfinder:task
Wayfinder child ticket: task
 wontfix
Will not be actioned
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
cameron
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Apps/ImageApi#72