http://bugzilla.moblin.org/show_bug.cgi?id=3427
--- Comment #16 from pohly <patrick.ohly(a)intel.com> 2009-08-25 00:22:12 ---
(In reply to comment #15)
I would name and describe the new configuration options a bit
differently:
"ResendTimeout",
"The time that the client waits before it assumes that\n"
"a message was lost and sends its own message again.\n"
"In addition, this timeout is used as delay between\n"
"a temporary network error and resending the failed\n"
"message."
"ResendRetries",
"How many times the client will resend a message.\n"
"When the maximum number of retries is exceeded,\n"
"the current session is automatically aborted without\n"
"sending further messages to the server."
Note the second sentence about resending after a temporary network problem. We
haven't discussed that yet and the current code doesn't implement it, does it?
I think it would be a useful reuse of the new error recovery approach. Does
this look useful/feasible?
Congwu implemented it like that. I just merged these changes into the master
trunk for 0.9.1.
Now we need to include the new "Client::Sync::*::Resend" test in the nightly
testing. My expectation is that it should work with all servers, but this needs
to be tested.
Congwu, can you work with Yongsheng on the integration into the nightly
testing? Is it possible to speed up the Resend tests by running them with
shorter timeouts?
--
Configure bugmail:
http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.