[jclouds] JCLOUDS-1065: Deprecate inDirectory option (#903)

[Andrew Gaul <no...@github.com>]

Superseded by prefix and delimiter support.
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/903

-- Commit Summary --

  * JCLOUDS-1065: Deprecate inDirectory option

-- File Changes --

    M blobstore/src/main/java/org/jclouds/blobstore/options/ListContainerOptions.java (10)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/903.patch
https://github.com/jclouds/jclouds/pull/903.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/903

Re: [jclouds] JCLOUDS-1065: Deprecate inDirectory option (#903)

[Andrew Gaul <no...@github.com>]

@timuralp Please review.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/903#issuecomment-173834958

Re: [jclouds] JCLOUDS-1065: Deprecate inDirectory option (#903)

[Andrew Gaul <no...@github.com>]

As a rule, jclouds only includes a feature in the portable abstraction when three providers support it.  Since only Swift and Atmos natively support directories and other providers are unlikely to support this odd feature, we should not expose it in the portable abstraction.  Callers can use the provider-specific interface if needed.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/903#issuecomment-174219856

Re: [jclouds] JCLOUDS-1065: Deprecate inDirectory option (#903)

[Timur Alperovich <no...@github.com>]

I like this change and believe we should be able to remove the directory notions out of jclouds!

Having said that, there are some providers where the behavior between delimiter/prefix and directory notions is not quite identical (I won't mention the file system provider, because it's pretty convoluted there anyway). In swift, the path parameter (what is used for directories) may return different results than when using delimiter/prefix. It supports the notion of directory blobs, which prefix and delimiter would not actually implicitly create. However, I don't know how many people actually rely on that functionality. I do think jclouds should deprecate support for directory blobs and move away from using the path parameter. None of the above precludes deprecating the support, but to actually switch over, consumers may need to do additional work on top of changing the parameters.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/903#issuecomment-174037917