Github user alopresto commented on the issue:
https://github.com/apache/nifi/pull/1530
Thanks @pvillard31 . For the record, I know some people prefer the Java 8
lambdas, and I come from a Ruby/Groovy background where I use closures and
chained collection operations for everything, but there can be pitfalls to
replacing concise traditional Java for:each loops with lambdas (see
[performance
issues](http://blog.takipi.com/benchmark-how-java-8-lambdas-and-streams-can-make-your-code-5-times-slower/)
and [other
concerns](https://stackoverflow.com/questions/16635398/java-8-iterable-foreach-vs-foreach-loop/20177092#20177092))
in addition to providing a learning curve for future developers who are not
familiar with the construct.
I'm not going to request changing this back, and after I do a final review
& smoke test, I'll merge this, but in the future I don't think we need to be
*as* aggressive about ensuring new code always uses the newest feature when the
existing process is fine.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
