@demobox Thanks for the feedback and sorry for my delayed responses; please see
the latest commit.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/514#issuecomment-87526600
@@ -254,4 +257,8 @@
*/
boolean blobExists(String container, String name);
+ /**
+* @throws ContainerNotFoundException if the container is not present.
+*/
+ void copyBlob(URI copySource, String toContainer, String toName,
OptionalCopyBlobOptions options);
Changed
+
+import java.util.Map;
+
+import com.google.common.base.Optional;
+import com.google.common.collect.ImmutableMap;
+
+/**
+ * @see a href=http://msdn.microsoft.com/en-us/library/dd894037.aspx; /
+ */
+// TODO:
+// x-ms-source-if-modified-since
+// x-ms-source-if-unmodified-since
+//
+package org.jclouds.azureblob.binders;
+
+import java.util.Map;
+
+import org.jclouds.azure.storage.reference.AzureStorageHeaders;
+import org.jclouds.azureblob.options.CopyBlobOptions;
+import org.jclouds.http.HttpRequest;
+import org.jclouds.rest.Binder;
+
+import
+public class BindAzureCopyOptionsToRequest implements Binder {
+ @Override
+ public R extends HttpRequest R bindToRequest(R request, Object input) {
+ OptionalCopyBlobOptions optional = (OptionalCopyBlobOptions) input;
+ if (!optional.isPresent()) {
+ return request;
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an AS IS BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ *
@@ -375,4 +380,23 @@ public void testBlockOperations() throws Exception {
assertEquals(1, blocks.getBlocks().get(2).getContentLength());
getApi().deleteContainer(blockContainer);
}
+
+ @Test(timeOut = 5 * 60 * 1000, dependsOnMethods = {
testCreateContainer,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.jclouds.azureblob.options;
+
+import java.util.Map;
+
+import
Thanks for the pull request but it's release week in jclouds and that means
it's time to clean up the PR queue. This PR will be over 6 months old as of
April 1. If you intend to continue work on it, please make a comment by April
2. Otherwise it will be closed on April 3.
---
Reply to this
sorry that unasyncing probably screwed up this branch. let me know if you want
a hand rebasing.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/514#issuecomment-57894560
+package org.jclouds.azureblob.binders;
+
+import java.util.Map;
+
+import org.jclouds.azure.storage.reference.AzureStorageHeaders;
+import org.jclouds.azureblob.options.CopyBlobOptions;
+import org.jclouds.http.HttpRequest;
+import org.jclouds.rest.Binder;
+
+import
+package org.jclouds.azureblob.binders;
+
+import java.util.Map;
+
+import org.jclouds.azure.storage.reference.AzureStorageHeaders;
+import org.jclouds.azureblob.options.CopyBlobOptions;
+import org.jclouds.http.HttpRequest;
+import org.jclouds.rest.Binder;
+
+import
+public class BindAzureCopyOptionsToRequest implements Binder {
+ @Override
+ public R extends HttpRequest R bindToRequest(R request, Object input) {
+ OptionalCopyBlobOptions optional = (OptionalCopyBlobOptions) input;
+ if (!optional.isPresent()) {
+ return request;
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an AS IS BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ *
+
+import java.util.Map;
+
+import com.google.common.base.Optional;
+import com.google.common.collect.ImmutableMap;
+
+/**
+ * @see a href=http://msdn.microsoft.com/en-us/library/dd894037.aspx; /
+ */
+// TODO:
+// x-ms-source-if-modified-since
+// x-ms-source-if-unmodified-since
+//
@@ -375,4 +380,23 @@ public void testBlockOperations() throws Exception {
assertEquals(1, blocks.getBlocks().get(2).getContentLength());
getApi().deleteContainer(blockContainer);
}
+
+ @Test(timeOut = 5 * 60 * 1000, dependsOnMethods = {
testCreateContainer,
@@ -325,4 +330,16 @@
ListenableFutureListBlobBlocksResponse
getBlockList(@PathParam(container)
@ParamValidators(ContainerNameValidator.class) String container,
@PathParam(name)
String name);
+ /**
+* @see
@@ -325,4 +330,16 @@
ListenableFutureListBlobBlocksResponse
getBlockList(@PathParam(container)
@ParamValidators(ContainerNameValidator.class) String container,
@PathParam(name)
String name);
+ /**
+* @see
@@ -325,4 +330,16 @@
ListenableFutureListBlobBlocksResponse
getBlockList(@PathParam(container)
@ParamValidators(ContainerNameValidator.class) String container,
@PathParam(name)
String name);
+ /**
+* @see
@@ -325,4 +330,16 @@
ListenableFutureListBlobBlocksResponse
getBlockList(@PathParam(container)
@ParamValidators(ContainerNameValidator.class) String container,
@PathParam(name)
String name);
+ /**
+* @see
Going with a URI-based copy source. This allows copies between different
accounts via signed URLs.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/514#issuecomment-55011739
[jclouds-pull-requests-java-6
#69](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/69/)
SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/514#issuecomment-55020580
[jclouds-pull-requests
#1158](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1158/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/514#issuecomment-55030602
[jclouds-pull-requests
#1155](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1155/) FAILURE
Looks like there's a problem with this pull request
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/514#issuecomment-54738511
[jclouds » jclouds
#1609](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1609/) FAILURE
Looks like there's a problem with this pull request
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
Presently lacks support for CopyBlobOptions. API reference:
http://msdn.microsoft.com/en-us/library/dd894037.aspx
You can merge this Pull Request by running:
git pull https://github.com/andrewgaul/jclouds azure-copy-blob
Or you can view, comment on it, or merge it online at:
[jclouds-pull-requests-java-6
#66](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/66/)
SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/514#issuecomment-54737736
@@ -325,4 +330,16 @@
ListenableFutureListBlobBlocksResponse
getBlockList(@PathParam(container)
@ParamValidators(ContainerNameValidator.class) String container,
@PathParam(name)
String name);
+ /**
+* @see
I am still digesting the variants of `CopyBlobOptions`; it seems like Azure and
S3 implement the same ETag variants and copy/replace user metadata operations.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/514#issuecomment-54737804
[jclouds-pull-requests-java-6
#67](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/67/)
SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/514#issuecomment-54737921
[jclouds » jclouds
#1608](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1608/) FAILURE
Looks like there's a problem with this pull request
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
31 matches
Mail list logo
