On Fri, Apr 11, 2008 at 06:11:23PM -0700, Bernard Li wrote:
> 
> I just found another issue with your initial patch

hope you don't mean r1230, which was committed to stable already after
careful testing.

> -- once the cookie
> is set, and you decide to change the authority tag in your gmetad.conf
> (with restart), it won't change the gridstack which is an unexpected
> behaviour (the user would need to manually delete the cookie, to be
> able to get the expected results).

that is not the only scenario where this problem will be triggered, IMHO
even more problematic from my original analysis of the patch was that no
attempt was ever made to make it backward compatible.

a user with a cookie with the old format won't be able to get his gridwalk
cookie processed correctly and will require it to be removed or reset, but
since this is a temporary cookie with 1 day TTL and this functionality is not
critical, it shouldn't matter.

as I mentioned as well, the logic around setting up the cookie was bogus and
the documentation obsolete, but those were also obviously not showstoppers
either.

> The simplest solution around this
> problem is to simply set the cookie all the time.

or since this is not a critical problem, let the users clear their cookies
after an intrusive reconfiguration, which will fix itself in 1 day anyway.

if a simpler solution is needed, making the cookie only valid for the session
(which will require the users to restart the browser to fix the problem) will
be simpler but I don't think this problem is critical enough or "unexpected"
to warrant more than that.

> Is there really nothing that can be done on the library side to get
> this working?  I don't see the same behaviour of exponential growth of
> cookie size with a typical browser like Firefox.

this seems more of a real problem than the previous one, can you elaborate?

Carlo

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Ganglia-developers mailing list
Ganglia-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ganglia-developers

Reply via email to