*** Submitted:
rpc: don't hang tests on failure
As suggested in previous CL. Also delete Accept method, also suggested earlier and not done.
R=dimitern, fwereade CC= https://codereview.appspot.com/7324046
https://codereview.appspot.com/7324046/diff/2001/rpc/rpc_test.go File rpc/rpc_test.go (right):
https://codereview.appspot.com/7324046/diff/2001/rpc/rpc_test.go#newcode277 rpc/rpc_test.go:277: case <-time.After(3 * time.Second): On 2013/02/13 12:56:35, fwereade wrote: > Can we reasonably drop this to, say, half a second?
why bother? the timeout only happens on test failure, and in that case 3 seconds isn't an issue.
https://codereview.appspot.com/7324046/
« Back to merge proposal
*** Submitted:
rpc: don't hang tests on failure
As suggested in previous CL.
Also delete Accept method, also suggested earlier
and not done.
R=dimitern, fwereade /codereview. appspot. com/7324046
CC=
https:/
https:/ /codereview. appspot. com/7324046/ diff/2001/ rpc/rpc_ test.go
File rpc/rpc_test.go (right):
https:/ /codereview. appspot. com/7324046/ diff/2001/ rpc/rpc_ test.go# newcode277 test.go: 277: case <-time.After(3 * time.Second):
rpc/rpc_
On 2013/02/13 12:56:35, fwereade wrote:
> Can we reasonably drop this to, say, half a second?
why bother? the timeout only happens on test failure, and in that case 3
seconds isn't an issue.
https:/ /codereview. appspot. com/7324046/