[JCBC-144] flush command needs to use RESTful flush for Couchbase Buckets Created: 11/Nov/12  Updated: 11/Dec/12  Resolved: 11/Dec/12

Status: Resolved
Project: Couchbase Java Client
Component/s: Core
Affects Version/s: None
Fix Version/s: 1.1.0
Security Level: Public

Type: New Feature Priority: Blocker
Reporter: Matt Ingenthron Assignee: Matt Ingenthron
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   
The current flush command needs to be connected to the RESTful flush.

 Comments   
Comment by Michael Nitschinger [ 12/Nov/12 ]
To my findings there are two ways to implement this feature:

- Reuse the BucketManager for this, since it already provides basic capabilities for flushing (but needs some extension).
- Reimplement the whole thing.

I would go with extending the BucketManager for this, since it would also keep the CouchbaseClient itself lean.

There is one thing that we need to decide upon: we can't just override the flush() method, because the returned value is different (we can't return an operation future with the current implementation). If we want to, we could "fake" it into one (but I doubt it makes sense). Therefore I propose a new method (flushBucket) which should be used with the couchbase client and we need to document that the old flush methods only work against memcached servers.
Comment by Michael Nitschinger [ 12/Nov/12 ]
Tracked here: http://review.couchbase.com/#/c/22445/
Comment by Matt Ingenthron [ 03/Dec/12 ]
Pushing out to release 1.1.0, as it's secondary functionality.
Comment by Matt Ingenthron [ 07/Dec/12 ]
http://review.couchbase.org/#/c/22445/
Generated at Mon Sep 01 21:36:17 CDT 2014 using JIRA 5.2.4#845-sha1:c9f4cc41abe72fb236945343a1f485c2c844dac9.