-
Notifications
You must be signed in to change notification settings - Fork 41.2k
Performance regression between RC1 and latest snapshots with large amount of configuration properties #21416
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
Comments
After #21394 was merged I see roughly the same numbers, so it didn't really improve things. |
Thanks for spotting this. |
Attempt to improve the performance of the `ConfigurationPropertySource` adapters `containsDescendantOf` method. The method now operates on arrays rather than iterators and reduces the inner for-loop when possible. See gh-21416
@dreis2211 I've pushed something that I think will make a difference. Could you try your sample app again and see if things have improved? |
@philwebb We're good. It's even better than in 2.2.7 now. Thanks a lot.
|
@dreis2211 Out of curiosity, did you do a local build to get the latest snapshot? Asking because I don't think repo.spring.io has this change yet due to a failed build. |
I did, @mbhave |
thanks for confirming! |
Hi,
after opening #21394 I thought I ran some benchmarks again on a project with a large amount of configuration properties. We've used https://github.com/martinvisser/config-properties-example in the past for this.
When doing so, I get the following results:
As you can see, the latest snapshot is about half a second slower than RC1. I'm not quite sure yet if this can be only traced back to the new hashCode calculation that was introduced in #20625, so I thought I better open an issue to investigate.
Cheers,
Christoph
The text was updated successfully, but these errors were encountered: