Introduce a filter and map imperative method to replace .stream() calls by bbakerman · Pull Request #3931 · graphql-java/graphql-java · GitHub
Skip to content

Introduce a filter and map imperative method to replace .stream() calls#3931

Merged
dondonz merged 2 commits intomasterfrom
imperative-filter-and-map
May 15, 2025
Merged

Introduce a filter and map imperative method to replace .stream() calls#3931
dondonz merged 2 commits intomasterfrom
imperative-filter-and-map

Conversation

@bbakerman
Copy link
Copy Markdown
Member

For performance reasons this replaces .stream() calls that build ImmutableLists

@bbakerman bbakerman added the performance work that is primarily targeted as performance improvements label Apr 24, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 24, 2025

@bbakerman bbakerman requested review from andimarek and dondonz April 27, 2025 06:25
@dondonz dondonz modified the milestones: 25.x breaking changes, 24.0 May 13, 2025
Copy link
Copy Markdown
Member

@dondonz dondonz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a separate note, given how many perf changes are in this release, we should look at the cumulative perf improvements later

…nd-map

# Conflicts:
#	src/main/java/graphql/util/FpKit.java
@dondonz dondonz merged commit b376b80 into master May 15, 2025
2 checks passed
@dondonz dondonz deleted the imperative-filter-and-map branch May 15, 2025 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance work that is primarily targeted as performance improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants