ctubbsii commented on PR #35: URL: https://github.com/apache/accumulo-classloaders/pull/35#issuecomment-3730951973
I was going to move to Guava Hashing instead of commons-codec DigestUtils, because it's faster, but it's a bit hard to work with the InputStreams directly without using the Beta HashingInputStream. I really would like to use the HashingInputStream, because it means we can compute the hash during the download of resources, rather than do a second read, but I'm not sure it's worth using a beta function, and also I'm not sure if it has as strong of a verification as doing the second read after it is written to disk. So, I'm not changing away from commons-codec. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
