Uploaded image for project: 'Giraph (Retired)'
  1. Giraph (Retired)
  2. GIRAPH-87

Simplify boolean expression in BspService::checkpointFrequencyMet

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.0.0
    • None
    • None

    Description

              if (superstep < firstCheckpoint) {
                  return false;
              } else if (((superstep - firstCheckpoint) % checkpointFrequency) == 0) {
                  return true;
              } else {
                  return false;
              }

      can be simplified to just return the result of the else if evaluation.

      Attachments

        1. GIRAPH-87.patch
          0.6 kB
          Eli Reisman
        2. GIRAPH-87.patch
          0.6 kB
          Eli Reisman

        Activity

          People

            initialcontext Eli Reisman
            jghoman Jakob Homan
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: