pub-dev icon indicating copy to clipboard operation
pub-dev copied to clipboard

pub.search.handlers: Slow search: handler exceeded 200ms

Open sortie opened this issue 7 years ago • 7 comments

I see a lot of these warnings in the search service logs. These messages obscure other warning level problems. We should either address the performance problem or demote the warning to a note.

However, dumping performance metrics to the logs is not the right solution. The right solution is to gather the performance data, plot that data, and monitor that instead and see if the probability distribution of latency is acceptable.

sortie avatar Aug 23 '18 09:08 sortie

The right solution is to gather the performance data, plot that data, and monitor that instead and see if the probability distribution of latency is acceptable.

Is there a ready-made solution we can just plug our data into?

isoos avatar Aug 23 '18 09:08 isoos

I'm sure there are, though I don't know them that well. Perhaps we can use things like google data studio for this?

sortie avatar Aug 23 '18 11:08 sortie

This is still a problem (even got worse?)

We need profiling data. Where is all that time spent?

  • Are we spending a lot of time loading package details for the listing?
    • these are in the worker output that potentially requires two fetches from cloud storage
      • should we preload this into datastore or redis?
  • Do we need to move search into the frontend service to reduce the time in communication between services?
  • How long time does the search itself take? (Can we create a dashboard for that)

sigurdm avatar Sep 21 '23 09:09 sigurdm

From initial investigations it seems that most of the time is spent in the search service.

sigurdm avatar Sep 21 '23 11:09 sigurdm

Still pending...

sigurdm avatar Jan 11 '24 09:01 sigurdm

We should try profiling the search service - maybe we can find ways to optimize. I think we should be able to handle much higher request rates than we are.

sigurdm avatar Jan 30 '24 15:01 sigurdm

Still an issue

sigurdm avatar Feb 29 '24 10:02 sigurdm