-
-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
performance degradation from 2023.7.1 to 2023.11.1 #31
Comments
There are essentially no functional changes to this library whatsoever between those two versions, so something else is happening. I'd need to see a minimal reproducer -- you've already profiled your code and seen it's this library specifically that's involved? |
This library is not a direct dependency of our code. It comes as a dependency of jsonschema (which is pinned in our dependencies to I can trigger the high CPU usage by doing
and bring things back to normal with
Nothing else changes in the meantime (no other sub-dependencies get re-installed by these commands) |
I see the issue, should be fixed in a moment with a new release. Thanks for the report. |
Hi team. We just upgraded from 2023.7.1 to 2023.11.1 and our CPU utilization increased by almost an order of magnitude with the same workload. We use this package indirectly via
jsonschema == 4.18.6
.Are you aware of what could cause such an issue. If yes, is there an easy workaround?
The affected line of code is here. In case it can help you with the investigation.
https://github.com/rucio/rucio/blob/0290623118d0890e799b848d72f2c1a3b18b2b04/lib/rucio/core/trace.py#L356C13-L356C21
The text was updated successfully, but these errors were encountered: