search

roysmith / spi-tools

8 stars 3 forks source link

UserActivitiesView should limit search depth #51

Open roysmith opened 4 years ago

roysmith commented 4 years ago

Imagine we ask for:

count = 10 main=False draft=True others=False

on a user with 1 million edits but none to draft space. We'll iterate through all 1 million edits trying to find 10 drafts. There should be some circuit breaker which cuts the search off before then. Maybe never examine more than 10*count?

roysmith commented 3 years ago

UserActivitiesView no longer exists, but the same issue applies to TimelineView.