giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alessandro Presta (JIRA)" <>
Subject [jira] [Created] (GIRAPH-537) Log message for aggregators has cryptic wording
Date Mon, 25 Feb 2013 06:42:12 GMT
Alessandro Presta created GIRAPH-537:

             Summary: Log message for aggregators has cryptic wording
                 Key: GIRAPH-537
             Project: Giraph
          Issue Type: Bug
            Reporter: Alessandro Presta
            Assignee: Maja Kabiljo

When workers are synchronizing aggregators using the ExpectedBarrier class, the following
message is printed:

"waitForRequiredPermits: Waiting for times required to be X (currently Y)"

I find this doesn't convey much about what exactly we're waiting for.

We should make the log more meaningful and explicitly mention aggregators, at the cost of
simplifying things a bit.
Additionally, we could skip it altogether when no aggregators are being used.

The ExpectedBarrier class itself also lacks clear documentation: what is this class for? The
Javadoc explains how it works without first stating what it is.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message