Apps/ImageApi
1
0
Fork 0
Code Issues Pull Requests 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
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
No items
No Label
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
Delete Branch "perf/faces-embeddings-no-clone"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?