@famous-airline-14628 - appreciate the honest pushback.
You're right to question this. I'll be upfront:
1.
AI disclosure: I used AI tools during my research. I should have stated that from the start per the contribution policy. My mistake.
2.
What I got wrong: The
_is_in_namespace N+1 pattern is intentional - there's a comment at
line 2660 of core.py that explicitly explains the design choice. I missed that and overstated it as a bug. Same with
_apply_filter - the
filters dict is already passed into
_get_object_internal, so server-side implementations can and likely do filter there. I was wrong to frame the client-side filter as the whole story.
3.
What I still stand by:
latest_successful_run iterates all runs and checks
run.successful one by one, which chains through
end_task → step → task → artifact lookup. With 10 consecutive failed runs, that's ~31 HTTP requests to find one successful run. This isn't a design choice with a code comment - it's just how the property was written. Whether it matters in practice at Netflix's scale, I can't say.
I'll narrow this issue to just point 3, or close it entirely if the maintainers don't find it useful. Either way - lesson learned on doing better due diligence before opening issues.