vvysotskyi commented on a change in pull request #1397: DRILL-6633: Replace
usage of Guava classes by JDK ones
URL: https://github.com/apache/drill/pull/1397#discussion_r210882276
##########
File path:
contrib/storage-mongo/src/test/java/org/apache/drill/exec/store/mongo/TestMongoChunkAssignment.java
##########
@@ -162,7 +163,7 @@ public void setUp() throws UnknownHostException {
@Test
public void testMongoGroupScanAssignmentMix() throws UnknownHostException,
ExecutionSetupException {
- final List<DrillbitEndpoint> endpoints = Lists.newArrayList();
+ final List<DrillbitEndpoint> endpoints = new ArrayList<>();
Review comment:
Let's concentrate on the changes which I did, since the project has a lot of
places, where code may be improved. Also, using `final` keyword is not critical
in general and may be used at the discretion of the developer in similar cases.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
