Skip to content
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

Reduce CPU cost of using java.util.Streams in Krystal code #169

Open
RamAnvesh opened this issue Aug 16, 2023 · 1 comment
Open

Reduce CPU cost of using java.util.Streams in Krystal code #169

RamAnvesh opened this issue Aug 16, 2023 · 1 comment
Assignees
Labels
performance Performance enhancement related

Comments

@RamAnvesh
Copy link
Collaborator

Rough analysis of CPU flame charts show significant usage of CPU by Stream pipelines
For example, in one instance, filtering a list by applying a condition and finding the first match is taking 5x more CPU than the condition itself.
This means out of all the streams being used in Krystal, upto 4x CPU is being wasted by Streams.
If this can be reclaimed by replacing streams with for loops over iterators, we should do that.

@RamAnvesh RamAnvesh added the performance Performance enhancement related label Aug 16, 2023
@RamAnvesh RamAnvesh added this to the v9 milestone Feb 2, 2025
@RamAnvesh
Copy link
Collaborator Author

We need to Validate and quantify the actual impact of this

@RamAnvesh RamAnvesh removed this from the v9 milestone Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance enhancement related
Projects
None yet
Development

No branches or pull requests

2 participants