Merge lp:~clint-fewbar/gearmand/fix-connection-failover into lp:gearmand/1.0
Proposed by
Clint Byrum
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 377 | ||||
Proposed branch: | lp:~clint-fewbar/gearmand/fix-connection-failover | ||||
Merge into: | lp:gearmand/1.0 | ||||
Diff against target: |
76 lines (+37/-6) 2 files modified
libgearman/connection.c (+6/-6) tests/worker_test.c (+31/-0) |
||||
To merge this branch: | bzr merge lp:~clint-fewbar/gearmand/fix-connection-failover | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brian Aker | Needs Information | ||
Review via email: mp+47438@code.launchpad.net |
Description of the change
Fixes order of operations when errors happen to allow proper failover to other servers during CONNECT phase.
To post a comment you must log in.
Why comment out these lines?
+ //gearman_return_t ret= gearman_ worker_ echo(cloned, "test", sizeof("test"));
62 +
63 + //test_truth(ret == GEARMAN_SUCCESS);