It breaks the /?filter=summary but that is easily fixed by including
r1215:

Index: server.c
===================================================================
--- server.c    (revision 1185)
+++ server.c    (revision 1215)
@@ -121,13 +121,9 @@
       source->hosts_up, source->hosts_down);
    if (rc) return 1;
 
-   pthread_mutex_unlock(root.sum_finished);
-   if (source->ds) {
-      pthread_mutex_lock(source->sum_finished);
-      rc = hash_foreach(source->metric_summary, metric_summary, (void*)
client);
-      pthread_mutex_unlock(source->sum_finished);
-   }
-   pthread_mutex_unlock(root.sum_finished);
+   pthread_mutex_lock(source->sum_finished);
+   rc = hash_foreach(source->metric_summary, metric_summary, (void*)
client);
+   pthread_mutex_unlock(source->sum_finished);
    
    return rc;
 }
 


If we add that, then I will vote for it since that is what I am running
in production now.  Thanks!

-twitham

>-----Original Message-----
>From: [EMAIL PROTECTED] [mailto:ganglia-
>[EMAIL PROTECTED] On Behalf Of Bernard Li
>Sent: Friday, April 11, 2008 2:02 PM
>To: Ganglia Developers
>Subject: [Ganglia-developers] [RFC] Backport #76 "gmetad summary
>dataincorrect during summarization" fixes to 3.0.x
>
>Hi guys:
>
>Another backport request.  This is already fixed in trunk, and I would
>like this to be backported to 3.0.x tree.  Please refer to the
>attached patch.
>
>Patch includes changes from revisions 1122, 1138, 1142, 1152, 1154 and
>1158 from trunk.
>
>Cheers,
>
>Bernard

-------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/ganglia-developers

Reply via email to