Callers should instead explicitly set contentMD5, usually with the
results from Guava Hashing.md5(). This narrows the API and removes a
strange IOException from callers.  Further it removes a dangerous
rebuffering of arbitrarily-large non-repeatable Payloads.
You can merge this Pull Request by running:

  git pull https://github.com/andrewgaul/jclouds remove-calculate-md5

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds/pull/381

-- Commit Summary --

  * Remove BlobBuilder and Payloads.calculateMD5

-- File Changes --

    M apis/atmos/src/test/java/org/jclouds/atmos/AtmosClientLiveTest.java (23)
    M 
apis/filesystem/src/main/java/org/jclouds/filesystem/strategy/internal/FilesystemStorageStrategyImpl.java
 (5)
    M 
apis/swift/src/test/java/org/jclouds/openstack/swift/CommonSwiftClientLiveTest.java
 (4)
    M blobstore/src/main/clojure/org/jclouds/blobstore2.clj (12)
    M 
blobstore/src/main/java/org/jclouds/blobstore/TransientStorageStrategy.java (10)
    M blobstore/src/main/java/org/jclouds/blobstore/domain/BlobBuilder.java (9)
    M 
blobstore/src/main/java/org/jclouds/blobstore/domain/internal/BlobBuilderImpl.java
 (10)
    M blobstore/src/test/clojure/org/jclouds/blobstore2_test.clj (9)
    M 
blobstore/src/test/java/org/jclouds/blobstore/TransientBlobRequestSignerTest.java
 (11)
    M 
blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java
 (7)
    M 
blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseContainerIntegrationTest.java
 (4)
    M core/src/main/java/org/jclouds/io/Payloads.java (40)
    M 
core/src/test/java/org/jclouds/rest/internal/RestAnnotationProcessorTest.java 
(26)
    M 
providers/azureblob/src/test/java/org/jclouds/azureblob/AzureBlobClientLiveTest.java
 (4)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/381.patch
https://github.com/jclouds/jclouds/pull/381.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/381

Reply via email to