Closed indianwhocodes closed 5 years ago
I believe that the approach specified in this PR is not a good approach. Buckets is a feature of the server-side API and should not require being enabled in the SDK. Adding this increases the burden placed upon the users of the SDK and doesn't provide any apparent benefit.
If users attempt to use buckets features against a Manta install that does not support them, then the SDK should error and ideally inform them that the Manta install doesn't support buckets.
Based on
I believe that the approach specified in this PR is not a good approach. Buckets is a feature of the server-side API and should not require being enabled in the SDK. Adding this increases the burden placed upon the users of the SDK and doesn't provide any apparent benefit.
If users attempt to use buckets features against a Manta install that does not support them, then the SDK should error and ideally inform them that the Manta install doesn't support buckets.
That makes sense, we should probably just configure the existing integration tests in such a way that it works both for buckets
and directories
.
I am gonna go ahead and close PR #536
Goals
Added a new
config
parameter for Manta Buckets Usage inJAVA SDK
.Setting
manta.buckets=true
\MANTA_BUCKETS=true
as anenv
variable will enable the client to use bucket operations enumerated as:createBucket
deleteBucket
listBucket
and so on.Naming convention for the said
config
parameter is chosen with the intent to make it consistent withenableBuckets
evident throughout the different components of manta including muskie, electric-boray, boray etc.This configuration parameter is also vital to running
integration-tests
for buckets, since the base path for running buckets will be/$MANTA_USER/buckets/
.DEFAULT_BUCKETS_ENABLED has been set to
false
since we don't want to have directory operations(API calls) conflicting with the Buckets API Calls.Pending
buckets-m1
.Implementation
config
parameter to the Java SDK can be found in DEVELOPING.md.Correctness
context
will override the defaults.