dart-lang / pub-dev

The pub.dev website
https://pub.dev
BSD 3-Clause "New" or "Revised" License
782 stars 147 forks source link

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

Open sortie opened 6 years ago

sortie commented 6 years ago

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.

isoos commented 6 years ago

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?

sortie commented 6 years ago

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

sigurdm commented 1 year ago

This is still a problem (even got worse?)

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

sigurdm commented 1 year ago

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

sigurdm commented 9 months ago

Still pending...

sigurdm commented 8 months ago

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 commented 7 months ago

Still an issue