[PCBC-135] Wrong parameter description for delete() Created: 08/Oct/12 Updated: 11/Dec/12 Resolved: 27/Nov/12 |
|
| Status: | Closed |
| Project: | Couchbase PHP client library |
| Component/s: | docs |
| Affects Version/s: | 1.0, 1.1.0-dp4 |
| Fix Version/s: | 1.0.5, 1.1.0 |
| Security Level: | Public |
| Type: | Bug | Priority: | Minor |
| Reporter: | Dan Andersen | Assignee: | Karen Zeller |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
The docs API for the delete method says: ( http://www.couchbase.com/docs/couchbase-sdk-php-1.1/api-reference-update.html#table-couchbase-sdk_php_delete )
$object->delete($key [, $expiry ]) But the expire should actually be cas, as in the source https://github.com/couchbase/php-ext-couchbase/blob/master/couchbase.c#L401 |
| Comments |
| Comment by Matt Ingenthron [ 08/Oct/12 ] |
| Please update this in both the 1.0 and 1.1 SDK docs. |
| Comment by Karen Zeller [ 27/Nov/12 ] |
|
Should be fixed with this change, I think: <argument inherit="cas" optional="yes"/> |
| Comment by Karen Zeller [ 27/Nov/12 ] |
|
should be the fix: <argument inherit="cas" optional="yes"/> |
[PCBC-81] User reports expiration is not working properly Created: 05/Jul/12 Updated: 18/Sep/12 Resolved: 18/Sep/12 |
|
| Status: | Closed |
| Project: | Couchbase PHP client library |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.0.5 |
| Security Level: | Public |
| Type: | Bug | Priority: | Major |
| Reporter: | Mike Wiederhold | Assignee: | Michael Jackson |
| Resolution: | Cannot Reproduce | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
http://www.couchbase.com/forums/thread/recent-issues-couchbase-eviction-php-sessions-and-expired-keys
|
| Comments |
| Comment by Mark Nunberg [ 18/Sep/12 ] |
| The user states that the pecl-memcached extension is being used. Expiration seems to work with php-ext-couchbase, so I'm closing this. |
[PCBC-72] getDelayed seems to return a Boolean not a "string (Binary object)" as the docs state Created: 13/Jun/12 Updated: 11/Dec/12 Resolved: 27/Nov/12 |
|
| Status: | Closed |
| Project: | Couchbase PHP client library |
| Component/s: | docs, library |
| Affects Version/s: | 1.0.3 |
| Fix Version/s: | 1.0.5, 1.1.0 |
| Security Level: | Public |
| Type: | Bug | Priority: | Critical |
| Reporter: | Benjamin Young | Assignee: | Karen Zeller |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
I'm not sure if this is a doc error or a code error, but the documentation for getDelayed [1] states it returns a "string (Binary object)" when in fact it returns a Boolean.
The return of a Boolean is likely the intended response as getDelayed implies the use of a callback. I've marked this issue as being related to both docs & library, though, in case I've missed something. [1] http://www.couchbase.com/docs/couchbase-sdk-php-1.0/api-reference-retrieve.html#table-couchbase-sdk_php_get-delayed |
| Comments |
| Comment by Matt Ingenthron [ 07/Nov/12 ] |
| Mark: Please determine if this issue is still valid and help me triage it into the appropriate release. I've assigned it to you for triage, not necessarily to fix it. |
| Comment by Matt Ingenthron [ 12/Nov/12 ] |
|
Trond: Since this may be just a matter of getting correct documentation, I've bumped the priority so we try to triage it and decide where it belongs. Please allocate just a few minutes to it when you can. If it's a documentation fix that's needed, let's pass it along to Karen. If you're unsure what was originally intended, it may make sense to look at pecl-memcached. |
| Comment by Trond Norbye [ 27/Nov/12 ] |
| According to the code (and the documentation for pecl::memcached:getDelayed) it should return a boolean value. |
| Comment by Karen Zeller [ 27/Nov/12 ] |
|
I think this fixes it: <dcall id="get-delayed" call="getDelayed" inherit="none"> <m_title> Get a key to be fetched later </m_title> <args> <argument inherit="keyn"/> <argument inherit="with_cas" optional="yes"/> <argument inherit="callback" optional="yes"/> </args> <return inherit="boolean"/> </dcall> |
| Comment by Karen Zeller [ 27/Nov/12 ] |
|
<dcall id="get-delayed" call="getDelayed" inherit="none"> <m_title> Get a key to be fetched later </m_title> <args> <argument inherit="keyn"/> <argument inherit="with_cas" optional="yes"/> <argument inherit="callback" optional="yes"/> </args> <return inherit="boolean"/> </dcall> |
[PCBC-71] implement addserver() Created: 08/Jun/12 Updated: 06/Sep/12 Resolved: 06/Sep/12 |
|
| Status: | Closed |
| Project: | Couchbase PHP client library |
| Component/s: | library |
| Affects Version/s: | 1.0.4 |
| Fix Version/s: | 1.0.5 |
| Security Level: | Public |
| Type: | Bug | Priority: | Major |
| Reporter: | Perry Krug | Assignee: | Jan Lehnardt |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Flagged: |
Release Note
|
| Description |
|
Just opening a bug since I didn't see one already...there's currently no "supported" way to add multiple servers to the Couchbase PHP client instance
|
| Comments |
| Comment by Michael Nitschinger [ 29/Aug/12 ] |
|
This should be supported through libcouchbase.
You can pass in more nodes as "node1:port;node2:port;node3:port". If you like to work with arrays: $servers = array('node1:port', 'node2:port'); $cb = new Couchbase(join(';', $servers)); |
| Comment by Mark Nunberg [ 06/Sep/12 ] |
| This is now possible through the ';'-delimited string, or the array constructor option. Once the client is connected, it learns about other servers from the REST JSON stream.. |
[PCBC-60] how to provide redundancy with setting up client not clear Created: 26/Mar/12 Updated: 07/Nov/12 Resolved: 07/Nov/12 |
|
| Status: | Closed |
| Project: | Couchbase PHP client library |
| Component/s: | docs |
| Affects Version/s: | 1.0.1 |
| Fix Version/s: | 1.0.5 |
| Security Level: | Public |
| Type: | Bug | Priority: | Major |
| 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 |
|
A user says they could not determine how to set up the PHP client for failover per the docs. I think that means there is confusion about what is actually happening when setting up the URLs.
From the user: i didn't find anything in the examples about adding multiple servers kelye @ 2:19 like new Couchbase ('10.1.1.1:1121, 10.1.1.2:1121'. .. etc 2:19 for failover 2:19 I've explained things, but perhaps we need to do more at the documentation level? |
| Comments |
| Comment by Sergey Avseyev [ 10/Apr/12 ] |
|
Currently PHP client uses libcouchbase as the backend. And there are two points where it can load the list of backup URLs.
1. Creation of the connection. https://github.com/couchbase/libcouchbase/blob/master/include/libcouchbase/couchbase.h#L72-L76 This means that you can specify a list of the nodes, where first will be used for immediate connection, others for list of the backup nodes. 2. On cluster upgrade, each time cluster push updated configuration, the connection instances save list of the known nodes, and if the initial node fails, it picks next from that list. In PHP client you can just use host parameter to pass the list of the nodes |
[PCBC-35] Add automated cluster tests Created: 03/Feb/12 Updated: 12/Nov/12 Resolved: 12/Nov/12 |
|
| Status: | Resolved |
| Project: | Couchbase PHP client library |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.0.5, 1.1.0 |
| Security Level: | Public |
| Type: | Improvement | Priority: | Major |
| Reporter: | Jan Lehnardt | Assignee: | Mark Nunberg |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Comments |
| Comment by Matt Ingenthron [ 07/Nov/12 ] |
| I believe this is closeable with the new testing framework you've built, correct? If you agree, please close. Let's look more at the roadmap on this one sometime soon. |
| Comment by Mark Nunberg [ 12/Nov/12 ] |
| Yes. We might want to make automated tests (i.e. 'make test'), but there's not much which is php-specific.. I'm imagining trond's build server is sufficient as far as these things go.. |