Details

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 2.3.0
    • None
    • Reviewed

    Description

      verifyBlockPlacement() has unused parameter -srcPath as its responsibility just verify single block rather than files under a specific path. Also the return value (int) does not make sense as the violation of block placement has other case than number of racks, so boolean value should be better.

      Attachments

        1. HDFS-4885.patch
          4 kB
          Junping Du
        2. HDFS-4885-v2.patch
          14 kB
          Junping Du
        3. HDFS-4885-v3.patch
          13 kB
          Junping Du
        4. HDFS-4885-v4.patch
          13 kB
          Junping Du
        5. HDFS-4885-v5.patch
          13 kB
          Junping Du
        6. HDFS-4885-v5.1.patch
          13 kB
          Junping Du

        Activity

          People

            junping_du Junping Du
            junping_du Junping Du
            Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: