[
https://issues.apache.org/jira/browse/JCLOUDS-545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14006039#comment-14006039
]
ASF subversion and git services commented on JCLOUDS-545:
---------------------------------------------------------
Commit b6c21f8ed1bfd48688a164d876e9eb6f2eaa39c4 in jclouds-labs-openstack's
branch refs/heads/1.7.x from [~jdaggett]
[
https://git-wip-us.apache.org/repos/asf?p=jclouds-labs-openstack.git;h=b6c21f8 ]
JCLOUDS-545: Use established conventions for OpenStack APIs, updates JavaDocs
and tests
Conflicts:
rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/features/CloudFilesCDNApiLiveTest.java
> SwiftApi and related Apis improvements
> --------------------------------------
>
> Key: JCLOUDS-545
> URL: https://issues.apache.org/jira/browse/JCLOUDS-545
> Project: jclouds
> Issue Type: Bug
> Components: jclouds-labs-openstack
> Affects Versions: 1.7.2
> Reporter: Everett Toews
> Assignee: Jeremy Daggett
> Fix For: 1.8.0, 1.7.3
>
>
> So far there are a number of issue with the Beta SwiftApi and related Apis.
> # All of the methods in SwiftApi are misnamed. Every single other top-level
> OpenStack API in jclouds is of the form *get* WhateverApi *For*
> Zone/RegionAndSomething.
> ## The methods to get the second-level APIs need to follow this form: get,
> and For must be used to be consistent with all of the other OpenStack APIs.
> ## configuredRegions() should be getConfiguredRegions()
> ## These name changes should be *added* to SwiftApi. The old methods
> deprecated (to be removed in 1.8) and point users to the new methods.
> # ObjectApi.replace() is not a good name. I understand the reasoning behind
> it but it's not intuitive for users looking to put/upload something. The
> semantics of put already say "If the URI refers to an already existing
> resource, it is modified". This method should be renamed put().
> ## This name change should be *added* to ObjectApi. The old method deprecated
> (to be removed in 1.8) and point users to the new method.
> # ObjectApi.replace() forces users to include metadata as the last parameter.
> This must be optional.
> # ObjectApi.get() forces users to include GetOptions as the last parameter.
> This must be optional.
> # ContainerApi.list() needs a comment that metadata is not returned and
> head() must be called to get metadata
> # ContainerApi.createIfAbsent() forces users to include options as the last
> parameter. This must be optional.
> # In all of the Swift domain objects toStringHelper("") is called with an
> empty String so no class name is given. This is inconsistent with the usage
> of toStringHelper("") everywhere else and doesn't provide as much
> information. It needs to be changed to toStringHelper().
--
This message was sent by Atlassian JIRA
(v6.2#6252)
