LL perf (Inventory): Filter checks walk the whole fetch queue per folder during the initial fetch #351

New issue

Closed

opened 2026-10-04 14:09:26 +00:00 by sky_line · 1 comment

sky_line commented

2026-10-04 14:09:26 +00:00

Owner

From the LL-origin inefficiency audit (finding I15). File/line references are from commit 0229f8872c; re-locate them before working.

Filter checks walk the whole fetch queue per folder during the initial fetch

Where: llinventoryfilter.cpp:202-204 → llinventorymodelbackgroundfetch.cpp:238-245, 1265-1283.
Also: mExpectedFolderIds is a std::list with a find per callback (:593, 640).
Confirmed. S.

Verify the cost (profile or timer) before and after changing it.

From the LL-origin inefficiency audit (finding I15). File/line references are from commit 0229f8872c; re-locate them before working. **Filter checks walk the whole fetch queue per folder during the initial fetch** - **Where:** `llinventoryfilter.cpp:202-204` → `llinventorymodelbackgroundfetch.cpp:238-245, 1265-1283`. - **Also:** `mExpectedFolderIds` is a `std::list` with a `find` per callback (`:593, 640`). - Confirmed. S. Verify the cost (profile or timer) before and after changing it.

sky_line added the

perf

label

2026-10-04 14:09:26 +00:00

sky_line added this to the Slipstream project

2026-10-06 15:04:13 +00:00

sky_line added the

status

in-progress

label

2026-10-08 02:07:13 +00:00

sky_line referenced this issue from a commit