steveloughran commented on a change in pull request #2604:
URL: https://github.com/apache/hadoop/pull/2604#discussion_r555707578
##########
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/AbfsOutputStream.java
##########
@@ -29,10 +29,10 @@
import java.util.concurrent.LinkedBlockingQueue;
import java.util.concurrent.ExecutorCompletionService;
import java.util.concurrent.ThreadPoolExecutor;
-import java.util.concurrent.Callable;
import java.util.concurrent.Future;
import java.util.concurrent.TimeUnit;
+import org.apache.hadoop.fs.statistics.StreamStatisticNames;
Review comment:
nit: needs to go into the real org.apache block. The IDE is getting
confused because of the third party stuff. as this is a "real" hadoop import it
should go down below; when we backport the google ones will go back to their
unshaded names
##########
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/AbfsOutputStreamStatisticsImpl.java
##########
@@ -18,32 +18,47 @@
package org.apache.hadoop.fs.azurebfs.services;
+import java.util.concurrent.atomic.AtomicLong;
+
+import org.apache.hadoop.fs.statistics.StreamStatisticNames;
Review comment:
move into real apache group.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
