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).
Hide
Permalink
Trond Norbye added a comment -
http://review.couchbase.org/23118
Show
Trond Norbye added a comment - http://review.couchbase.org/23118

People

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

Dates

  • Created:
    Updated:
    Resolved: