Couchbase PHP client library

make default view timeout 75s

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 1.1.0-dp1, 1.1.0-dp2, 1.1.0-dp3, 1.1.0-dp4, 1.1.0-dp5
  • Fix Version/s: 1.1.0
  • Component/s: None
  • Security Level: Public
  • Labels:
    None

Description

Since the server side view timeout is 60s, we should raise the client side a bit higher. 75s seems like the right number.

It must also be tuneable through the constructor to the client or the .ini file or both.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Matt Ingenthron added a comment -
Per discussion with Trond, we don't currently do anything for timeout other than the default libcouchbase timeout. On this issue, we'll change the timeout to the 75s standard.

For what it's worth, it wouldn't hurt my feelings if we were to create a second timeout in libcouchbase for http requests. Maybe there is one already? The only problem with that is that the API/ABI in libcouchbase is required to be stable for now (since we hit Beta).
Show
Matt Ingenthron added a comment - Per discussion with Trond, we don't currently do anything for timeout other than the default libcouchbase timeout. On this issue, we'll change the timeout to the 75s standard. For what it's worth, it wouldn't hurt my feelings if we were to create a second timeout in libcouchbase for http requests. Maybe there is one already? The only problem with that is that the API/ABI in libcouchbase is required to be stable for now (since we hit Beta).
Matt Ingenthron made changes -
Field Original Value New Value
Summary Raise view timeout from 60s to 75s make default view timeout 75s
Description Since the server side view timeout is 60s, we should raise the client side a bit higher. 75s seems like the right number. It should be tuneable though. Since the server side view timeout is 60s, we should raise the client side a bit higher. 75s seems like the right number.

It must also be tuneable through the constructor to the client or the .ini file or both.
Trond Norbye made changes -
Status Open [ 1 ] In Progress [ 3 ]
Hide
Permalink
Trond Norbye added a comment -
http://review.couchbase.org/23118
Show
Trond Norbye added a comment - http://review.couchbase.org/23118
Matt Ingenthron made changes -
Fix Version/s 1.1.0 [ 10242 ]
Fix Version/s 1.1.0-dp6 [ 10402 ]
Matt Ingenthron made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]

People

  • Assignee:
    Trond Norbye
    Reporter:
    Matt Ingenthron
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: