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

Author	SHA1	Message	Date
Cameron Cordes	fb4df4b195	style: cargo fmt sweep Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-01 19:01:00 -04:00
Cameron Cordes	1d9b9a0bc4	faces: avoid 40 MB row clone in /faces/embeddings 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>	2026-05-01 19:00:55 -04:00

Author

SHA1

Message

Date

Cameron Cordes

fb4df4b195

style: cargo fmt sweep

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

2026-05-01 19:01:00 -04:00

Cameron Cordes

1d9b9a0bc4

faces: avoid 40 MB row clone in /faces/embeddings

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>

2026-05-01 19:00:55 -04:00

perf/faces-embeddings-no-clone #72

2 Commits