Why do we have so many calls to saveReceivedEvent but not processSavedEvent?

planetary-social / nos-notification-service-go

A service which delivers push notifications to APNs

Mozilla Public License 2.0

3 stars 0 forks source link

Why do we have so many calls to saveReceivedEvent but not processSavedEvent? #33

Closed boreq closed 1 year ago

boreq commented 1 year ago

Possibly related:

https://github.com/planetary-social/nos-notification-service-go/pull/34

boreq commented 1 year ago

This is most likely due to the fact that we reconnect to relays and grab the same events over and over again. They end up being passed to the "save event handler" but not to the "process saved event handler" as they are determined to be dupes. I can create a fix to reduce load on the Firestore by using a memory cache.

boreq commented 1 year ago

https://github.com/planetary-social/nos-notification-service-go/pull/35

boreq commented 1 year ago

Let's close this I don't think this is a bug as I said.