Skip to content

Use Java 8 Stream in stream and aggregateStream Template methods #3944

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.

Sign up for GitHub

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

Jump to bottom
Closed
mp911de opened this issue Jan 24, 2022 · 0 comments
Closed

Use Java 8 Stream in stream and aggregateStream Template methods #3944

mp911de opened this issue Jan 24, 2022 · 0 comments
Assignees
@mp911de
Labels
type: enhancement A general enhancement
Milestone
4.0 M2 (2022.0.0)

Comments

@mp911de
Copy link
Member

mp911de commented Jan 24, 2022

With the usage of Java 17 as baseline we want to switch to Java 8 streams directly to avoid the indirection through CloseableIterator. We also need to document that the stream needs to be closed (e.g. to be used with try-with-resources).
@mp911de mp911de added the type: enhancement A general enhancement label Jan 24, 2022
@mp911de mp911de self-assigned this Jan 24, 2022
@mp911de mp911de mentioned this issue Jan 24, 2022
Closed
christophstrobl pushed a commit that referenced this issue Feb 3, 2022
@mp911de @christophstrobl
Use Java 8 Stream as return type for Template operations returning a …
2367379 
…stream.

We now use Stream instead of CloseableIterator for easier stream creation.

Closes: #3944
Original Pull Request: #3946
@christophstrobl christophstrobl added this to the 4.0 M2 (2022.0.0) milestone Feb 3, 2022
@christophstrobl christophstrobl linked a pull request Feb 3, 2022 that will close this issue
Use Java 8Stream as return type for Template operations returning a stream #3946
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mp911de mp911de

Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
4.0 M2 (2022.0.0)
2 participants
@mp911de @christophstrobl