@vdiravka The purpose of `org.apache.drill.common.collections.Collectors` is to use internal iterator with a functional interface instead of creating a stream. In many cases, I'd recommend to use `Collectors` and extend this class. Using streams is justified when processing thousands of elements, for a handful number of elements, `Collectors` is a better option, IMO.
@vvysotskyi Personally, I don't see why it is necessary to replace *all* Guava classes/methods with provided by JDK. IMO, the scope of PR should be to replace only removed and deprecated by Guava team API, so it can be reduced significantly. I don't see why it is necessary to change `ImmutableEntry` class as part of this PR when it needs to be completely replaced with the one provided by JDK. [ Full content available at: https://github.com/apache/drill/pull/1397 ] This message was relayed via gitbox.apache.org for [email protected]
