You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by ga...@apache.org on 2014/07/19 01:59:49 UTC
git commit: JCLOUDS-457: Fix complete MPU archive size
Repository: jclouds-labs-aws
Updated Branches:
refs/heads/master 6b952c49e -> f7d2319ce
JCLOUDS-457: Fix complete MPU archive size
The completeMultipartUpload operation is taking the archive size
parameter in MB but it should be bytes.
Project: http://git-wip-us.apache.org/repos/asf/jclouds-labs-aws/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds-labs-aws/commit/f7d2319c
Tree: http://git-wip-us.apache.org/repos/asf/jclouds-labs-aws/tree/f7d2319c
Diff: http://git-wip-us.apache.org/repos/asf/jclouds-labs-aws/diff/f7d2319c
Branch: refs/heads/master
Commit: f7d2319ce04950ed989d2751407173c0f58af645
Parents: 6b952c4
Author: Roman Coedo <ro...@gmail.com>
Authored: Sun Jul 13 12:32:23 2014 +0200
Committer: Andrew Gaul <ga...@apache.org>
Committed: Fri Jul 18 16:57:21 2014 -0700
----------------------------------------------------------------------
.../src/main/java/org/jclouds/glacier/GlacierAsyncClient.java | 2 +-
glacier/src/main/java/org/jclouds/glacier/GlacierClient.java | 4 ++--
.../org/jclouds/glacier/binders/BindArchiveSizeToHeaders.java | 4 ++--
.../src/test/java/org/jclouds/glacier/GlacierClientMockTest.java | 2 +-
4 files changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/jclouds-labs-aws/blob/f7d2319c/glacier/src/main/java/org/jclouds/glacier/GlacierAsyncClient.java
----------------------------------------------------------------------
diff --git a/glacier/src/main/java/org/jclouds/glacier/GlacierAsyncClient.java b/glacier/src/main/java/org/jclouds/glacier/GlacierAsyncClient.java
index a1c1939..c7893ed 100644
--- a/glacier/src/main/java/org/jclouds/glacier/GlacierAsyncClient.java
+++ b/glacier/src/main/java/org/jclouds/glacier/GlacierAsyncClient.java
@@ -218,7 +218,7 @@ public interface GlacierAsyncClient extends Closeable {
@ParamValidators(VaultNameValidator.class) @PathParam("vault") String vaultName,
@PathParam("uploadId") String uploadId,
@BinderParam(BindMultipartTreeHashToHeaders.class) Map<Integer, HashCode> hashes,
- @BinderParam(BindArchiveSizeToHeaders.class) long archiveSizeInMB);
+ @BinderParam(BindArchiveSizeToHeaders.class) long archiveSize);
/**
* @see GlacierClient#abortMultipartUpload
http://git-wip-us.apache.org/repos/asf/jclouds-labs-aws/blob/f7d2319c/glacier/src/main/java/org/jclouds/glacier/GlacierClient.java
----------------------------------------------------------------------
diff --git a/glacier/src/main/java/org/jclouds/glacier/GlacierClient.java b/glacier/src/main/java/org/jclouds/glacier/GlacierClient.java
index 7826370..0960d0a 100644
--- a/glacier/src/main/java/org/jclouds/glacier/GlacierClient.java
+++ b/glacier/src/main/java/org/jclouds/glacier/GlacierClient.java
@@ -168,12 +168,12 @@ public interface GlacierClient extends Closeable {
* Multipart upload identifier.
* @param hashes
* Map containing the pairs partnumber-treehash of each uploaded part.
- * @param archiveSizeInMB
+ * @param archiveSize
* Size of the complete archive.
* @return A String containing the Archive identifier in Amazon Glacier.
* @see <a href="http://docs.aws.amazon.com/amazonglacier/latest/dev/api-multipart-complete-upload.html" />
*/
- String completeMultipartUpload(String vaultName, String uploadId, Map<Integer, HashCode> hashes, long archiveSizeInMB);
+ String completeMultipartUpload(String vaultName, String uploadId, Map<Integer, HashCode> hashes, long archiveSize);
/**
* Aborts the multipart upload.
http://git-wip-us.apache.org/repos/asf/jclouds-labs-aws/blob/f7d2319c/glacier/src/main/java/org/jclouds/glacier/binders/BindArchiveSizeToHeaders.java
----------------------------------------------------------------------
diff --git a/glacier/src/main/java/org/jclouds/glacier/binders/BindArchiveSizeToHeaders.java b/glacier/src/main/java/org/jclouds/glacier/binders/BindArchiveSizeToHeaders.java
index 38d5f87..b042051 100644
--- a/glacier/src/main/java/org/jclouds/glacier/binders/BindArchiveSizeToHeaders.java
+++ b/glacier/src/main/java/org/jclouds/glacier/binders/BindArchiveSizeToHeaders.java
@@ -32,9 +32,9 @@ public class BindArchiveSizeToHeaders implements Binder {
public <R extends HttpRequest> R bindToRequest(R request, Object input) {
checkArgument(input instanceof Long, "This binder is only valid for long");
checkNotNull(request, "request");
- Long archiveSizeInMB = Long.class.cast(input);
+ Long archiveSize = Long.class.cast(input);
return (R) request.toBuilder()
- .replaceHeader(GlacierHeaders.ARCHIVE_SIZE, Long.toString(archiveSizeInMB << 20))
+ .replaceHeader(GlacierHeaders.ARCHIVE_SIZE, Long.toString(archiveSize))
.build();
}
http://git-wip-us.apache.org/repos/asf/jclouds-labs-aws/blob/f7d2319c/glacier/src/test/java/org/jclouds/glacier/GlacierClientMockTest.java
----------------------------------------------------------------------
diff --git a/glacier/src/test/java/org/jclouds/glacier/GlacierClientMockTest.java b/glacier/src/test/java/org/jclouds/glacier/GlacierClientMockTest.java
index b53771b..19dda1c 100644
--- a/glacier/src/test/java/org/jclouds/glacier/GlacierClientMockTest.java
+++ b/glacier/src/test/java/org/jclouds/glacier/GlacierClientMockTest.java
@@ -310,7 +310,7 @@ public class GlacierClientMockTest {
2, partHashcode,
3, partHashcode,
4, partHashcode);
- assertThat(client.completeMultipartUpload(VAULT_NAME, MULTIPART_UPLOAD_ID, map, 8L)).isEqualTo(ARCHIVE_ID);
+ assertThat(client.completeMultipartUpload(VAULT_NAME, MULTIPART_UPLOAD_ID, map, 8 * MiB)).isEqualTo(ARCHIVE_ID);
RecordedRequest request = server.takeRequest();
assertEquals(request.getRequestLine(),