You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@libcloud.apache.org by "Tomaz Muraus (JIRA)" <ji...@apache.org> on 2012/12/22 19:23:12 UTC

[dev] [jira] [Comment Edited] (LIBCLOUD-271) Supporting KTUCloud Storage Driver.

    [ https://issues.apache.org/jira/browse/LIBCLOUD-271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13538871#comment-13538871 ] 

Tomaz Muraus edited comment on LIBCLOUD-271 at 12/22/12 6:22 PM:
-----------------------------------------------------------------

Some initial feedback after a quick review:

- Majority of the code inside the driver and tests seems to be copy and pasted from the OpenStack code and CloudFiles driver and tests. Copy and pasting is bad because it makes code hard to maintain, etc. so this should be improved by adding more common classes and re-using stuff where possible.
                
      was (Author: kami):
    Thanks for the patch!

Some initial feedback after a quick review:

- A lot of stuff inside the driver and tests seems to be copy and pasted from the  CloudFiles driver and tests. Copy and pasting is bad because it makes code hard to maintain, etc. so this should be improved by adding more common classes and re-using stuff where possible.
                  
> Supporting KTUCloud Storage Driver.
> -----------------------------------
>
>                 Key: LIBCLOUD-271
>                 URL: https://issues.apache.org/jira/browse/LIBCLOUD-271
>             Project: Libcloud
>          Issue Type: Improvement
>          Components: Storage
>            Reporter: DaeMyung Kang
>            Priority: Minor
>         Attachments: libcloud-271.diff
>
>
> Supporting KTUCloud Storage Driver.
> KTUCloud Storage is based on OpenStack(CloudFiles)
> But KTUCloud Storage is somewhat different from original OpenStack Swift.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira