[jclouds/jclouds] Fix for JIRA 1246 (#1068)

2017-03-01 Thread archupsg03
https://issues.apache.org/jira/browse/JCLOUDS-1246 You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/1068 -- Commit Summary -- * Next development version 2.0.1-SNAPSHOT * Avoid creating temporary files via relative paths * Include par

[jclouds/jclouds] Fix for JIRA 1246 (#1069)

2017-03-01 Thread archupsg03
https://issues.apache.org/jira/browse/JCLOUDS-1246 You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/1069 -- Commit Summary -- * Fix for JIRA 1246 -- File Changes -- M apis/openstack-swift/src/main/java/org/jclouds/openstack/swift/

Re: [jclouds/jclouds] Fix for JIRA 1246 (#1068)

2017-03-01 Thread archupsg03
Closed #1068. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1068#event-981755688

Re: [jclouds/jclouds] Allow extending SwiftBlobStoreContextModule (#1069)

2017-03-01 Thread archupsg03
My pleasure :) RegardsArchana On Thursday, 2 March 2017, 3:24, Ignasi Barrera wrote: Thanks for the contribution @archupsg03!— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread. -- You are receiving this

[jclouds/jclouds] JCLOUDS 1250 - GetBlob Issue (#1070)

2017-03-02 Thread archupsg03
https://issues.apache.org/jira/browse/JCLOUDS-1250 You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/1070 -- Commit Summary -- * JCLOUDS 1250 - GetBlob Issue -- File Changes -- M apis/openstack-swift/src/main/java/org/jclouds/opens

Re: [jclouds/jclouds] JCLOUDS 1250 - GetBlob Issue (#1070)

2017-03-02 Thread archupsg03
Closed #1070. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1070#event-983473260

[jclouds/jclouds] DLO Swift Implementation (#1090)

2017-04-28 Thread archupsg03
Have implemented Swift Dynamic Large Object upload strategy. Requires changes in 1. Manifest upload with X-Object-Manifest header 2. Remove blobs, in order to remove the manifest and segments You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pul

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-01 Thread archupsg03
archupsg03 commented on this pull request. > +* @param containerAndKey +* @return array of size 2 with container and objectPrefix +*/ + private String[] splitContainerAndKey(String containerAndKey) { + String[] parts = containerAndKey.split("/", 2); +

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-01 Thread archupsg03
archupsg03 commented on this pull request. > @@ -123,6 +124,50 @@ public void uploadMultipartBlob() { // The etag returned by Swift is not the md5 of the Blob uploaded // It is the md5 of the concatenated segment md5s } + + @Test + public void uploadMultipartBlob

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-01 Thread archupsg03
archupsg03 commented on this pull request. > + assertReplaceManifest(regionId, defaultContainerName, defaultName); + } + } + + protected void assertReplaceManifest(String regionId, String containerName, String name) { + ObjectApi objectApi = getApi().getObjectApi(regio

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-01 Thread archupsg03
archupsg03 commented on this pull request. >} - return api.getContainerApi(regionId).create(container, BASIC_CONTAINER); + resultContainerCreate = api.getContainerApi(regionId).create(container, BASIC_CONTAINER); + if (resultContainerCreate) { + Container

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-02 Thread archupsg03
For DLO upload, etag will the md5 of the manifest and not the content of the file thats getting uploaded. Thats the reason etag mismatch happens in ParallelLiveTest -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://git

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-02 Thread archupsg03
archupsg03 commented on this pull request. > + * 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 the License is distributed on an &qu

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-02 Thread archupsg03
archupsg03 commented on this pull request. >return api.getStaticLargeObjectApi(regionId, > mpu.containerName()).replaceManifest(mpu.blobName(), builder.build(), mpu.blobMetadata().getUserMetadata(), getContentMetadataForManifest(mpu.blobMetadata().getContentMe

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-03 Thread archupsg03
archupsg03 commented on this pull request. > +* @param containerAndKey +* @return array of size 2 with container and objectPrefix +*/ + private String[] splitContainerAndKey(String containerAndKey) { + String[] parts = containerAndKey.split("/", 2); +

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-03 Thread archupsg03
archupsg03 commented on this pull request. > + * 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 the License is distributed on an &qu

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-18 Thread archupsg03
archupsg03 commented on this pull request. > +* @param prefix +* +*/ + private void removeObjectsWithPrefix(String container, String prefix) { + String nextMarker = null; + ObjectApi objectApi = api.getObjectApi(regionId, container); +

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-19 Thread archupsg03
archupsg03 commented on this pull request. > + * 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 the License is distributed on an &qu

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-19 Thread archupsg03
@archupsg03 pushed 1 commit. c6f17c1 Comments Address - Except separation of higher and lower level abstraction -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1090/files/13bb2016abc6abb2a6d25fdf8dcb4c3517f692ca

[jclouds/jclouds] JIRA_1250 (#1104)

2017-05-19 Thread archupsg03
You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/1104 -- Commit Summary -- * JIRA Defect 1250, containerExist check -- File Changes -- M apis/openstack-swift/src/main/java/org/jclouds/openstack/swift/v1/blobstore/RegionScopedSwif

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-19 Thread archupsg03
@archupsg03 pushed 1 commit. e4eff0d Build failure issue addressed -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1090/files/297610fe9120f6e37b33810c2f673c3c5a256a99..e4eff0dc0c167f6a3d4bdc654d4d9a12d85bbdd8

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-19 Thread archupsg03
@archupsg03 pushed 1 commit. 2fe1ef3 Build failure issue addressed -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1090/files/e4eff0dc0c167f6a3d4bdc654d4d9a12d85bbdd8..2fe1ef35fd4c65f6602b0f0b80466f6550e61331

Re: [jclouds/jclouds] JIRA-1250 (#1104)

2017-05-21 Thread archupsg03
@archupsg03 pushed 1 commit. 465dc0e Unit test for JIRA 1250 -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1104/files/da7aa12354bfe30227292c508a6f1ac40a291ead..465dc0ef9a932b3a6ded6b7835c4211d07e483dc

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-22 Thread archupsg03
@archupsg03 pushed 1 commit. c1c338f Provider Changes -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1090/files/2fe1ef35fd4c65f6602b0f0b80466f6550e61331..c1c338f2d029441a894b87b55c8654f730360e5b

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-22 Thread archupsg03
No changes are required in RegionScopedBlobStore. Working on the DynamicLargeObjectAPI. Will submit PR for that. Closing this PR. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1090#i

Re: [jclouds/jclouds] JClouds 1281- DLO Swift Implementation (#1090)

2017-05-22 Thread archupsg03
Closed #1090. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1090#event-1092913378

Re: [jclouds/jclouds] JIRA-1250 (#1104)

2017-05-22 Thread archupsg03
archupsg03 commented on this pull request. > } + @Test(expectedExceptions = { IllegalStateException.class }) + public void getBlobTestFailure() { + blobStore.containerExists(CONTAINER_TEMP); Problem is with getBlob and not containerCreation. -- You are receiving this beca

[jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-23 Thread archupsg03
You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/1105 -- Commit Summary -- * DLO - Lower Level Provider API changes -- File Changes -- M apis/openstack-swift/src/main/java/org/jclouds/openstack/swift/v1/SwiftApi.java (6) A

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-23 Thread archupsg03
@archupsg03 pushed 1 commit. 0dff0ae DLO - Comments Added -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/28ec26db280a6efdf17a9085bbc6d2efb47e00a7..0dff0ae919f6c11550300940aba5f3b7e96f7a77

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-23 Thread archupsg03
archupsg03 commented on this pull request. > +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefixing/escaping the header +* name/key. +* +* @return {@l

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-23 Thread archupsg03
archupsg03 commented on this pull request. > + @BinderParam(SetPayload.class) Payload blob, + @BinderParam(BindObjectMetadataToHeaders.class) Map metadata, + @BinderParam(BindToHeaders.class) Map headers); + + /** +* Delete objects. +* +* @param objectN

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-23 Thread archupsg03
archupsg03 commented on this pull request. > + +import com.google.common.collect.ImmutableMap; +import com.google.common.collect.ImmutableMap.Builder; + +/** + * Updates Content Metadata. + * + * Required to update the Content Length of Object File as Sum of Content length + * of Segme

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
@archupsg03 pushed 1 commit. 9906c00 DLO - Comments Addressed1 -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/0dff0ae919f6c11550300940aba5f3b7e96f7a77..9906c00c68933634126014e60f1bfd05ade7f897

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefixing/escaping the header +* name/key. +* +* @return {@l

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > +* ordered parts which will be concatenated upon download. +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefix

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > +* @param objectName +* corresponds to {@link SwiftObject#getName()}. +* @param segments +* ordered parts which will be concatenated upon download. +* @param metadata +* corresponds to {@l

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
@archupsg03 pushed 1 commit. b8c04a0 DLO - method name change -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/9906c00c68933634126014e60f1bfd05ade7f897..b8c04a07eae6abd285d13f04d32b0c1ae4a6b2a4

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > + @BinderParam(SetPayload.class) Payload blob, + @BinderParam(BindObjectMetadataToHeaders.class) Map metadata, + @BinderParam(BindToHeaders.class) Map headers); + + /** +* Delete objects. +* +* @param objectN

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > + +import com.google.common.base.Function; +import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableMap; +import com.google.common.collect.Lists; +import com.google.common.io.ByteSource; +imp

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > + @SuppressWarnings("deprecation") + @Test + public void uploadLargeFile(String regionId) throws IOException, InterruptedException { + /* + * createRandomFile(SIZE, bigFile); Payload payLoad = new + * FilePa

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > + @Test + public void testReplaceManifest() throws Exception { + for (String regionId : regions) { + assertReplaceManifest(regionId, defaultContainerName, defaultName); + uploadLargeFile(regionId); + removeLargeF

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefixing/escaping the header +* name/key. +* +* @return {@l

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > + String objName = objectName + "/dlo/" + String.valueOf(partNumber); + String data = "data".concat(String.valueOf(partNumber)); + String etag = getApi().getDynamicLargeObjectApi(regionId

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > + // configure the blobstore to use multipart uploading of the file + /* + * payLoad.getContentMetadata().setContentLength(bigFile.length()); + * BasePayloadSlicer slicer = new BasePayloadSlicer(); + */ + wh

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > + @Test + public void uploadLargeFile(String regionId) throws IOException, InterruptedException { + /* + * createRandomFile(SIZE, bigFile); Payload payLoad = new + * FilePayload(bigFile); + */ + List list1 =

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > + assertEquals(bigObject.getETag(), etagOfEtags); + assertEquals(bigObject.getPayload().getContentMetadata().getContentLength(), Long.valueOf(2 * 1024 * 1024)); + assertEquals(bigObject.getMetadata(), ImmutableMap.of("myfo

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-24 Thread archupsg03
archupsg03 commented on this pull request. > +getApi().getObjectApi(regionId, containerName).delete(name); + } + } + + @Override + @BeforeClass(groups = "live") + public void setup() { + super.setup(); + for (String regionId : regions) { +

Re: [jclouds/jclouds] JIRA-1250 (#1104)

2017-05-24 Thread archupsg03
@archupsg03 pushed 1 commit. 7070073 Jira 1250 - Minor issue in code -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1104/files/465dc0ef9a932b3a6ded6b7835c4211d07e483dc..7070073f64da941c6a926ab7baa549d452ff9afa

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-29 Thread archupsg03
archupsg03 commented on this pull request. > +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefixing/escaping the header +* name/key. +* +* @return {@l

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-29 Thread archupsg03
archupsg03 commented on this pull request. > +* ordered parts which will be concatenated upon download. +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefix

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-29 Thread archupsg03
archupsg03 commented on this pull request. > + for (int i = partNumber; i < 3; partNumber++) { + String objName = String.format("%s/%s/%s", objectName, "dlo", partNumber); + String data = String.format("%s%s", "data"

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-29 Thread archupsg03
archupsg03 commented on this pull request. > + + awaitConsistency(); + + SwiftObject bigObject = getApi().getObjectApi(regionId, containerName).get(name); + assertThat(bigObject.getETag().equals(etagOfEtags)); + assertThat(bigObject.getPayload().getContentMetad

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-29 Thread archupsg03
archupsg03 commented on this pull request. > + // segments are visible + assertThat(getApi().getContainerApi(regionId).get(containerName).getObjectCount().equals(Long.valueOf(3))); + } + + protected void assertMegabyteAndETagMatches(String regionId, String containerName, Str

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-29 Thread archupsg03
archupsg03 commented on this pull request. > + SwiftObject object1s = getApi().getObjectApi(regionId, > containerName).get(name); + assertThat(object1s.getETag().equals(etag1s)); + assertThat(object1s.getPayload().getContentMetadata().getContentLength().equals(Long.valueO

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-29 Thread archupsg03
archupsg03 commented on this pull request. > + @Override + @BeforeClass(groups = "live") + public void setup() { + super.setup(); + for (String regionId : regions) { + boolean created = getApi().getContainerApi(regionId).create(defaultContainerName);

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-29 Thread archupsg03
archupsg03 commented on this pull request. > + public void setup() { + super.setup(); + for (String regionId : regions) { + boolean created = getApi().getContainerApi(regionId).create(defaultContainerName); + if (!created) { +deleteAllObjectsInContai

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-05-29 Thread archupsg03
archupsg03 commented on this pull request. > + super.setup(); + for (String regionId : regions) { + boolean created = getApi().getContainerApi(regionId).create(defaultContainerName); + if (!created) { +deleteAllObjectsInContainer(regio

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-04 Thread archupsg03
archupsg03 commented on this pull request. > +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefixing/escaping the header +* name/key. +* +* @return {@l

Re: [jclouds/jclouds] JIRA-1250 (#1104)

2017-06-04 Thread archupsg03
archupsg03 commented on this pull request. > } + @Test(expectedExceptions = { IllegalStateException.class }) + public void getBlobTestFailure() { + blobStore.containerExists(CONTAINER_TEMP); @andrewgaul Can you look into this, there was a minor mistake previously -- You

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-05 Thread archupsg03
archupsg03 commented on this pull request. > +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefixing/escaping the header +* name/key. +* +* @return {@l

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-06 Thread archupsg03
archupsg03 commented on this pull request. > + +import com.google.common.collect.ImmutableMap; +import com.google.common.collect.ImmutableMap.Builder; + +/** + * Updates Content Metadata. + * + * Required to update the Content Length of Object File as Sum of Content length + * of Segme

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-06 Thread archupsg03
archupsg03 commented on this pull request. > +import com.google.common.collect.ImmutableMap.Builder; + +/** + * Updates Content Metadata. + * + * Required to update the Content Length of Object File as Sum of Content length + * of Segments + * + * @param contentMetadata + * + * @return

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-06 Thread archupsg03
archupsg03 commented on this pull request. > + * use it and give us feedback. Based on that feedback, minor changes to the + * interfaces may happen. This code will replace + * org.jclouds.openstack.swift.SwiftClient in jclouds 2.0 and it is recommended + * you adopt it sooner than la

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-06 Thread archupsg03
@archupsg03 pushed 1 commit. 998b6a9 DLO Manifest Comments addressed -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/b8c04a07eae6abd285d13f04d32b0c1ae4a6b2a4..998b6a926c52797549393348fdf44f7b0da3d557

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-06 Thread archupsg03
archupsg03 commented on this pull request. > +* checksum of the concatenated ETag values of the {@code > segments}. +*/ + @Deprecated + @Named("dynamicLargeObject:replaceManifest") + @PUT + @ResponseParser(ETagHeader.class) + @Headers(keys = &q

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-06 Thread archupsg03
archupsg03 commented on this pull request. > + @Deprecated + @Named("dynamicLargeObject:replaceManifest") + @PUT + @ResponseParser(ETagHeader.class) + @Headers(keys = "X-Object-Manifest", values = "{containerName}/{objectName}/") + String repla

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-06 Thread archupsg03
archupsg03 commented on this pull request. > +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefixing/escaping the header +* name/key. +* +* @return {@l

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-07 Thread archupsg03
archupsg03 commented on this pull request. > +* @param metadata +* corresponds to {@link SwiftObject#getMetadata()}. +* @param headers +* Binds the map to headers, without prefixing/escaping the header +* name/key. +* +* @return {@l

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-07 Thread archupsg03
archupsg03 commented on this pull request. > + @PUT + @ResponseParser(ETagHeader.class) + @Headers(keys = "X-Object-Manifest", values = "{containerName}/{objectName}/") + String putManifest(@PathParam("objectName") String o

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-07 Thread archupsg03
archupsg03 commented on this pull request. > +@RequestFilters(AuthenticateRequest.class) +@Consumes(APPLICATION_JSON) +@Path("/{objectName}") +public interface DynamicLargeObjectApi { + /** +* Creates or updates a dynamic large object's manifest. +* +*

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-07 Thread archupsg03
archupsg03 commented on this pull request. > + > assertThat(getApi().getContainerApi(regionId).get(defaultContainerName).getObjectCount()).isEqualTo(Long.valueOf(3)); + } + + @SuppressWarnings("deprecation") + protected void assertReplaceManifest(String

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-15 Thread archupsg03
@archupsg03 pushed 1 commit. 05ba514 Made testcase work for latest changes -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/998b6a926c52797549393348fdf44f7b0da3d557

Re: [jclouds/jclouds] JIRA-1250 (#1104)

2017-06-15 Thread archupsg03
archupsg03 commented on this pull request. > } + @Test(expectedExceptions = { IllegalStateException.class }) + public void getBlobTestFailure() { + blobStore.containerExists(CONTAINER_TEMP); @andrewgaul Any further comments on this ? -- You are receiving this because you

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-15 Thread archupsg03
@archupsg03 pushed 1 commit. 2fba009 Minor change -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/b1ea32f70e1469d50b0e7298015a304c62f258f4..2fba0091268ac08364fb045d474b5ae0435b3301

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-15 Thread archupsg03
@timuralp Done with almost all the changes. can you review once -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1105#issuecomment-308945363

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-16 Thread archupsg03
@archupsg03 pushed 1 commit. 93b41da CheckStyle Exceptions -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/2fba0091268ac08364fb045d474b5ae0435b3301..93b41da44f42710ab1ed55a78ea05c88b5b2e7e1

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-06-26 Thread archupsg03
Thanks @timuralp . Were you able to run test sucessfully ? Sure we ll wait for @andrewgaul -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1105#issuecomment-311243633

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-07-02 Thread archupsg03
@archupsg03 pushed 1 commit. 3aa1552 Null Pointer Exception in DLO Live Test fixed -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/93b41da44f42710ab1ed55a78ea05c88b5b2e7e1

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-07-02 Thread archupsg03
@timuralp Have fixed the NPE exception and tests run successfully for me. Can you give a try ? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1105#issuecomment-312554318

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-07-06 Thread archupsg03
@timuralp Am not facing any checkstyle exception in the final commit. Can you just point out to few white space -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1105#issuecomment-31335

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-07-09 Thread archupsg03
@archupsg03 pushed 1 commit. 42f0273 Removed Trailing white spaces -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/3aa1552342bf030eef0838a43cc354cff7dcc7b9..42f0273b6c704ee3b544ffce10266aae9dd40d72

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-07-09 Thread archupsg03
@andrewgaul Removed trailing whitespaces that was visible to me :(. If you are still spacing any issues I think its better you can change accordingly. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclou

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-07-20 Thread archupsg03
@andrewgaul @timuralp any other blockers for merging this PR -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1105#issuecomment-316906273

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-08-23 Thread archupsg03
@andrewgaul . Sure will work on it by this week end. One more clarification here, incase of swift MPU say you have 3 parts and due to some reason if the uploading of one part failed, current behaviour is it tries continuing with rest of the parts. But expectation is not to proceed if there is a

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-08-25 Thread archupsg03
archupsg03 commented on this pull request. > + +import org.jclouds.io.Payloads; +import org.jclouds.openstack.swift.v1.SwiftApi; +import org.jclouds.openstack.v2_0.internal.BaseOpenStackMockTest; +import org.testng.annotations.Test; + +import com.google.common.collect.ImmutableMap; +imp

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-08-25 Thread archupsg03
archupsg03 commented on this pull request. > +@Consumes(APPLICATION_JSON) +@Path("/{objectName}") +public interface DynamicLargeObjectApi { + /** +* Creates or updates a dynamic large object's manifest. +* +* @param objectName +* corresponds to

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-08-28 Thread archupsg03
@archupsg03 pushed 1 commit. 2ee67e1 Minor comments addressed -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1105/files/42f0273b6c704ee3b544ffce10266aae9dd40d72..2ee67e1ef0030f223fb7e31ccf58e9eb1e46d768

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-08-28 Thread archupsg03
archupsg03 commented on this pull request. > + +import org.jclouds.io.Payloads; +import org.jclouds.openstack.swift.v1.SwiftApi; +import org.jclouds.openstack.v2_0.internal.BaseOpenStackMockTest; +import org.testng.annotations.Test; + +import com.google.common.collect.ImmutableMap; +imp

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-08-29 Thread archupsg03
Line number 87 in DynamicLargetObjectApiLiveTest is call to ObjectApi.put methods which was existing and this is a basic call. It may be because of some other issue -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://gi

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-09-06 Thread archupsg03
Regarding the rackspace issue, I am not sure of it, and its a basic and existing call. @timuralp Any comment on this -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1105#issuecomment-

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-09-21 Thread archupsg03
@andrewgaul, Since we are not working with rackspace cloud, we dont have a setup to test against it. If possible can you share rackspace cloud credentials, so that I can try to figure out the issue -- You are receiving this because you are subscribed to this thread. Reply to this email directly

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-09-21 Thread archupsg03
ok no problem. But can you check whether the issue is because of one of the 2 Jira's mentioned in above comment. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1105#issuecomment-33134

Re: [jclouds/jclouds] DLO - Lower Level Provider API changes (#1105)

2017-09-21 Thread archupsg03
I am doubting that container doesnot exist in rackspace cloud, so why we are facing the issue. Can you make sure that the container used for DLO exist and then give a try -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: htt