Giraph
  1. Giraph
  2. GIRAPH-173

BspCase:getNumWorkers javadoc refers to non-existent parameter

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Not a Problem
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

        /**
         * Get the number of workers used in the BSP application
         *
         * @param numProcs number of processes to use
         */
        public int getNumWorkers() {
          return numWorkers;
        }

      numProcs is a lie...

      1. GIRAPH-173.patch
        0.4 kB
        Abhishek Srivastava

        Activity

        Jakob Homan created issue -
        Hide
        Abhishek Srivastava added a comment -

        Fixed broken comment for getNumWorkers.

        Testing Done:

        • mvn compile
        • mvn test
        Show
        Abhishek Srivastava added a comment - Fixed broken comment for getNumWorkers. Testing Done: mvn compile mvn test
        Abhishek Srivastava made changes -
        Field Original Value New Value
        Attachment GIRAPH-173.patch [ 12524111 ]
        Avery Ching made changes -
        Assignee Abhishek Srivastava [ asrivas ]
        Hide
        Avery Ching added a comment -

        This was fixed before your patch was reviewed Abhishek. Thanks for going through the process though. Please feel free to look at other issues.

        Show
        Avery Ching added a comment - This was fixed before your patch was reviewed Abhishek. Thanks for going through the process though. Please feel free to look at other issues.
        Avery Ching made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Not A Problem [ 8 ]

          People

          • Assignee:
            Abhishek Srivastava
            Reporter:
            Jakob Homan
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development