steveloughran commented on a change in pull request #2350:
URL: https://github.com/apache/hadoop/pull/2350#discussion_r525370313
##########
File path:
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/compress/lz4/Lz4Decompressor.java
##########
@@ -22,8 +22,9 @@
import java.nio.Buffer;
import java.nio.ByteBuffer;
+import net.jpountz.lz4.LZ4Factory;
+import net.jpountz.lz4.LZ4SafeDecompressor;
Review comment:
yeah, if things are mixed up, best to leave alone -at least for those
files which get lots of changes. For something which rarely sees maintenance,
you can make a stronger case for cleanup. I do it sometimes, but as I also get
to field cherrypick merge pain, I don't go wild on it
----------------------------------------------------------------
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]
