django-eventstream icon indicating copy to clipboard operation
django-eventstream copied to clipboard

Published 20 hours ago • verified publisher icon fanout

trim_event_log inefficient

Open Schlepptop opened this issue 3 years ago • 1 comments

Is there a reason to not use a single delete statement when trimming the event log? The code creates a query for every single event that needs to be deleted. I can create a PR if that's unintentional.

Schlepptop avatar Jun 12 '22 19:06 Schlepptop

I think it's unintentional. There's already EVENT_TRIM_BATCH to limit the size of the bulk deletes. PR welcome. :)

jkarneges avatar Jun 14 '22 01:06 jkarneges