Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2769#discussion_r204755463
--- Diff:
storm-server/src/main/java/org/apache/storm/utils/ServerUtils.java ---
@@ -635,16 +635,16 @@ public static void unZip(File inFile, File toDir)
throws IOException {
*
* @throws IOException
*/
+ //Used only by logviewer, considering moving to web-app?
--- End diff --
even if used only in one place, if it is a general purpose function, I
think it makes sense to have in a global place. I'd remove this comment.
---
