On Jun 3, 2008, at 1:39 PM, Carlo Marcelo Arenas Belon wrote:
> On Tue, Jun 03, 2008 at 11:36:55AM +0200, Sebastien Piechurski wrote:
>> Here is the same patch made against revision 1373 of the svn
>> repository.
>
> just one minor caveat, for consistency (not that it will make any
> difference
> though, as the value is overloaded anyway), the default from
> libgmond should
> match the one from gmond.c and from the embedded configuration
> (copied excerpt
> below from the patch, as it was not possible to reply to your patch
> directly
> because it was attached and not inlined)
>
Oops, missed this one. I did not catch the second value was the
default value for the variable ....
> Index: lib/libgmond.c
> ===================================================================
> --- lib/libgmond.c (revision 1375)
> +++ lib/libgmond.c (working copy)
> @@ -57,6 +57,7 @@
> CFG_BOOL("mute", 0, CFGF_NONE),
> CFG_BOOL("deaf", 0, CFGF_NONE),
> CFG_INT("host_dmax", 0, CFGF_NONE),
> + CFG_INT("host_tmax", 20, CFGF_NONE),
> CFG_INT("cleanup_threshold", 300, CFGF_NONE),
> CFG_BOOL("gexec", 0, CFGF_NONE),
> CFG_INT("send_metadata_interval", 0, CFGF_NONE),
>
>> I also added some explanations to the documentation, but am not sure
>> the explanation is really clear nor exact.
>
> it is correct, if a little strange, but will let any native speaker
> correct
> the grammar if needed.
>
> Committed revision 1376
>
> I also suspect that it will be probably better to also change the
> logic for
> gmond so that it will use the host_tmax value directly instead of a
> multiple
> of it with a hardcoded "4 attempts" to avoid confusion, and most
> likely also
> add some verification and documentation in the relationship that
> should be
> maintained between this value and the heartbeat, to avoid broken
> setups.
>
In this case, it would probably be better to change the name of the
configurable option to something more explicit, like for example
"host_down_timeout" (really just a suggestion). I assumed that "tmax"
meant "timeout max", but was not even sure of it, so host_tmax is
probably not the best name to be used, but I could not think of
somehitng else.
> Carlo
>
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Ganglia-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ganglia-developers
