jerrypeng commented on a change in pull request #11343:
URL: https://github.com/apache/pulsar/pull/11343#discussion_r671430151



##########
File path: 
pulsar-common/src/main/java/org/apache/pulsar/common/nar/NarUnpacker.java
##########
@@ -136,27 +146,27 @@ private static void makeFile(final InputStream 
inputStream, final File file) thr
     }
 
     /**
-     * Calculates an md5 sum of the specified file.
+     * Calculates an sha256 sum of the specified file.

Review comment:
       What is the reason to use SHA256 instead of md5sum?  The SHA hashing 
algorithm is much more resource intensive than md5




-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to