Skip to content

Polish: map entries should be iterated when both the key and value are needed #1752

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

Closed
wants to merge 1 commit into from

Conversation

igor-suhorukov
Copy link
Contributor

use java 8 language features to iterate on maps and transform data set by streams

@jhoeller
Copy link
Contributor

Inspired by this PR, I've taken a broad sweep over existing Map.Entry iteration cases as well: https://jira.spring.io/browse/SPR-16646, rolling your changes into it in a slightly revised fashion.

Once again, thanks for your efforts there!

@jhoeller jhoeller closed this Mar 26, 2018
@igor-suhorukov
Copy link
Contributor Author

@jhoeller thank you for your great work!
Could you please talk two words about Russian community contribution and my PRs on JPoint conference Moscow?

Regards,
Igor

@jhoeller
Copy link
Contributor

Absolutely, my JPoint keynote is about Spring Framework 5 and our JDK 8/9 specific work, and many of your contributions were related to Java 8+ API usage... a fine example for how community contributions make our ongoing theme much more complete that we initially shipped it, and how those contributions provide direct triggers for follow-up work on our side. It'll be my pleasure to explicitly mention your PRs!

@sbrannen
Copy link
Member

sbrannen commented Mar 27, 2018

@igor-suhorukov, you're like a... PR machine! 👏

@igor-suhorukov
Copy link
Contributor Author

@jhoeller thanks a lot for your time because of sometimes my pull requests are too noisy)))
I am waiting for your keynote in Moscow!

@igor-suhorukov
Copy link
Contributor Author

@sbrannen thanks) to be continued...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants