I added partial support for openstack-swift and reported
[JCLOUDS-872](https://issues.apache.org/jira/browse/JCLOUDS-872) for the
remaining features. Unfortunately Swift itself does not have the conditional
copy options that other providers offer. We may want to implement this with
two Swift
> @@ -138,6 +140,12 @@
> ListenableFuture putBlob(String container, Blob blob, PutOptions
> options);
>
> /**
> +* @see BlobStore#copyBlob(String,String,String,String)
> +*/
> + ListenableFuture copyBlob(String fromContainer, String fromName,
> String toContainer, String toNam
> @@ -224,6 +227,16 @@
> String putBlob(String container, Blob blob, PutOptions options);
>
> /**
> +* Copy blob from one container to another. Some providers implement this
> +* more efficiently than corresponding getBlob and putBlob operations.
Done.
---
Reply to this email d
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 emai
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/511#issuecomment-57894565
> @@ -138,6 +140,12 @@
> ListenableFuture putBlob(String container, Blob blob, PutOptions
> options);
>
> /**
> +* @see BlobStore#copyBlob(String,String,String,String)
> +*/
> + ListenableFuture copyBlob(String fromContainer, String fromName,
> String toContainer, String toNam
This operation is supported in native GCS API and should be easy to implenment .
https://developers.google.com/storage/docs/json_api/v1/objects/copy
I think overiding metadata options would be useful and with option method can
be used for it (if other providers support similar approach)?
---
Rep
> @@ -307,4 +314,31 @@ public boolean apply(String in) {
> */
> protected abstract boolean deleteAndVerifyContainerGone(String container);
>
> + @Override
> + public ListenableFuture copyBlob(String fromContainer, String
> fromName, String toContainer, String toName,
> + Opt
What kind of options do you have in mind @andrewgaul? I can make this work with
glacier.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/511#issuecomment-54723119
> @@ -307,4 +314,31 @@ public boolean apply(String in) {
> */
> protected abstract boolean deleteAndVerifyContainerGone(String container);
>
> + @Override
> + public ListenableFuture copyBlob(String fromContainer, String
> fromName, String toContainer, String toName,
> + Opt
> @@ -307,4 +314,31 @@ public boolean apply(String in) {
> */
> protected abstract boolean deleteAndVerifyContainerGone(String container);
>
> + @Override
> + public ListenableFuture copyBlob(String fromContainer, String
> fromName, String toContainer, String toName,
> + Opt
> + * (the "License"); you may not use this file except in compliance with
> + * the License. You may obtain a copy of the License at
> + *
> + * http://www.apache.org/licenses/LICENSE-2.0
> + *
> + * Unless required by applicable law or agreed to in writing, software
> + * distributed under t
> + * (the "License"); you may not use this file except in compliance with
> + * the License. You may obtain a copy of the License at
> + *
> + * http://www.apache.org/licenses/LICENSE-2.0
> + *
> + * Unless required by applicable law or agreed to in writing, software
> + * distributed under t
> @@ -307,4 +314,31 @@ public boolean apply(String in) {
> */
> protected abstract boolean deleteAndVerifyContainerGone(String container);
>
> + @Override
> + public ListenableFuture copyBlob(String fromContainer, String
> fromName, String toContainer, String toName,
> + Opt
> @@ -307,4 +314,31 @@ public boolean apply(String in) {
> */
> protected abstract boolean deleteAndVerifyContainerGone(String container);
>
> + @Override
> + public ListenableFuture copyBlob(String fromContainer, String
> fromName, String toContainer, String toName,
> + Opt
> @@ -224,6 +227,16 @@
> String putBlob(String container, Blob blob, PutOptions options);
>
> /**
> +* Copy blob from one container to another. Some providers implement this
> +* more efficiently than corresponding getBlob and putBlob operations.
Document that options are curren
> @@ -235,6 +237,13 @@ public String putBlob(String container, Blob blob,
> PutOptions overrides) {
>return sync.putObject(container, blob2Object.apply(blob), options);
> }
>
> + @Override
> + public String copyBlob(String fromContainer, String fromName, String
> toContainer, St
> @@ -234,6 +236,19 @@ public BlobMetadata apply(ObjectMetadata from) {
> }
>
> @Override
> + public ListenableFuture copyBlob(String fromContainer, String
> fromName, String toContainer, String toName,
> + Optional options) {
> + // TODO: honor options
In that case, perh
[jclouds ยป jclouds
#1605](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1605/) ABORTED
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/511#issuecomment-54703854
[jclouds-pull-requests
#1152](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1152/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/511#issuecomment-54702667
[jclouds-pull-requests-java-6
#63](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/63/)
SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/511#issuecomment-54701370
@jdaggett @shrinandj @hsbhathiya @rcoedo I appreciate feedback on what other
providers can support, especially for `CopyOptions`.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/511#issuecomment-54701352
You can merge this Pull Request by running:
git pull https://github.com/andrewgaul/jclouds copy-blob
Or you can view, comment on it, or merge it online at:
https://github.com/jclouds/jclouds/pull/511
-- Commit Summary --
* JCLOUDS-651: Add copyBlob to portable abstraction
* JCLOUDS-65
23 matches
Mail list logo