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 2013/07/17 19:37:47 UTC
[2/3] git commit: Cleanup of Azureblob code;
as per pull request review. Additional items not covered here are
assigned specific bug ids.
Cleanup of Azureblob code; as per pull request review. Additional items
not covered here are assigned specific bug ids.
Project: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/commit/a6f6f04d
Tree: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/tree/a6f6f04d
Diff: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/diff/a6f6f04d
Branch: refs/heads/1.6.x
Commit: a6f6f04d753dc8d681b1f0f4969159b0ca699152
Parents: 3a734b8
Author: John Kew <jo...@socrata.com>
Authored: Thu Jul 11 16:12:07 2013 -0700
Committer: Andrew Gaul <ga...@apache.org>
Committed: Wed Jul 17 10:35:18 2013 -0700
----------------------------------------------------------------------
.../blobstore/strategy/AzureBlobBlockUploadStrategy.java | 5 +++--
.../azureblob/domain/internal/BlobBlockPropertiesImpl.java | 3 ++-
.../azureblob/predicates/validators/BlockIdValidator.java | 4 ++--
.../org/jclouds/azureblob/xml/BlobBlocksResultsHandler.java | 2 ++
4 files changed, 9 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/a6f6f04d/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/strategy/AzureBlobBlockUploadStrategy.java
----------------------------------------------------------------------
diff --git a/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/strategy/AzureBlobBlockUploadStrategy.java b/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/strategy/AzureBlobBlockUploadStrategy.java
index d725875..4475555 100644
--- a/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/strategy/AzureBlobBlockUploadStrategy.java
+++ b/providers/azureblob/src/main/java/org/jclouds/azureblob/blobstore/strategy/AzureBlobBlockUploadStrategy.java
@@ -35,6 +35,7 @@ import java.util.List;
import static com.google.common.base.Preconditions.checkArgument;
import static com.google.common.base.Preconditions.checkNotNull;
+import static com.google.common.base.Preconditions.checkState;
/**
* Decomposes a blob into blocks for upload and assembly through PutBlock and PutBlockList
@@ -57,7 +58,7 @@ public class AzureBlobBlockUploadStrategy implements MultipartUploadStrategy {
public String execute(String container, Blob blob) {
String blobName = blob.getMetadata().getName();
Payload payload = blob.getPayload();
- long length = payload.getContentMetadata().getContentLength();
+ Long length = payload.getContentMetadata().getContentLength();
checkNotNull(length,
"please invoke payload.getContentMetadata().setContentLength(length) prior to azure block upload");
checkArgument(length <= (MAX_NUMBER_OF_BLOCKS * MAX_BLOCK_SIZE));
@@ -81,7 +82,7 @@ public class AzureBlobBlockUploadStrategy implements MultipartUploadStrategy {
blockIds.add(blockId);
client.putBlock(container, blobName, blockId, block);
}
- assert bytesWritten == length;
+ checkState(bytesWritten == length, "Wrote " + bytesWritten + " bytes, but we wanted to write " + length + " bytes");
return client.putBlockList(container, blobName, blockIds);
}
}
http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/a6f6f04d/providers/azureblob/src/main/java/org/jclouds/azureblob/domain/internal/BlobBlockPropertiesImpl.java
----------------------------------------------------------------------
diff --git a/providers/azureblob/src/main/java/org/jclouds/azureblob/domain/internal/BlobBlockPropertiesImpl.java b/providers/azureblob/src/main/java/org/jclouds/azureblob/domain/internal/BlobBlockPropertiesImpl.java
index 7e412f9..b2d15d7 100755
--- a/providers/azureblob/src/main/java/org/jclouds/azureblob/domain/internal/BlobBlockPropertiesImpl.java
+++ b/providers/azureblob/src/main/java/org/jclouds/azureblob/domain/internal/BlobBlockPropertiesImpl.java
@@ -19,6 +19,7 @@ package org.jclouds.azureblob.domain.internal;
import org.jclouds.azureblob.domain.BlobBlockProperties;
import com.google.common.base.Objects;
+import static com.google.common.base.Preconditions.checkNotNull;
/**
* Representation of the blocks which compose a Blob
@@ -29,7 +30,7 @@ public class BlobBlockPropertiesImpl implements BlobBlockProperties {
private final boolean committed;
public BlobBlockPropertiesImpl(String blockName, long contentLength, boolean committed) {
- this.blockName = blockName;
+ this.blockName = checkNotNull(blockName);
this.contentLength = contentLength;
this.committed = committed;
}
http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/a6f6f04d/providers/azureblob/src/main/java/org/jclouds/azureblob/predicates/validators/BlockIdValidator.java
----------------------------------------------------------------------
diff --git a/providers/azureblob/src/main/java/org/jclouds/azureblob/predicates/validators/BlockIdValidator.java b/providers/azureblob/src/main/java/org/jclouds/azureblob/predicates/validators/BlockIdValidator.java
index 9003f87..cd6d53f 100644
--- a/providers/azureblob/src/main/java/org/jclouds/azureblob/predicates/validators/BlockIdValidator.java
+++ b/providers/azureblob/src/main/java/org/jclouds/azureblob/predicates/validators/BlockIdValidator.java
@@ -33,8 +33,8 @@ import org.jclouds.predicates.Validator;
@Singleton
public class BlockIdValidator extends Validator<String> {
@Override
- public void validate(@Nullable String s) throws IllegalArgumentException {
- if (s.length() > 64)
+ public void validate(String s) throws IllegalArgumentException {
+ if (s == null || s.length() > 64)
throw new IllegalArgumentException("block id:" + s + "; Block Ids must be less than or equal to 64 bytes in size");
}
http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/a6f6f04d/providers/azureblob/src/main/java/org/jclouds/azureblob/xml/BlobBlocksResultsHandler.java
----------------------------------------------------------------------
diff --git a/providers/azureblob/src/main/java/org/jclouds/azureblob/xml/BlobBlocksResultsHandler.java b/providers/azureblob/src/main/java/org/jclouds/azureblob/xml/BlobBlocksResultsHandler.java
index 4a95065..6a566e7 100755
--- a/providers/azureblob/src/main/java/org/jclouds/azureblob/xml/BlobBlocksResultsHandler.java
+++ b/providers/azureblob/src/main/java/org/jclouds/azureblob/xml/BlobBlocksResultsHandler.java
@@ -30,6 +30,7 @@ import com.google.common.collect.Lists;
/**
* Parses the following document:
+ * <pre>
* <?xml version="1.0" encoding="utf-8"?>
* <BlockList>
* <CommittedBlocks>
@@ -39,6 +40,7 @@ import com.google.common.collect.Lists;
* </Block>
* <CommittedBlocks>
* </BlockList>
+ * </pre>
*/
public class BlobBlocksResultsHandler extends ParseSax.HandlerWithResult<ListBlobBlocksResponse> {