We should remove ```calculateMD5``` because it violates the principle of least 
astonishment.  For repeatable payloads like ```InputSupplier``` it incurs IO 
*twice* and for non-repeatable payloads like ```InputStream``` it *buffers* the 
entire bolus in-memory, often times causing ```OutOfMemoryError```.  We should 
let the user see the cost, e.g., calling Files.hash, rather than taking magical 
actions on the user's behalf.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/218#issuecomment-29779783

Reply via email to