Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: HDFS XAttrs (HDFS-2006)
    • Fix Version/s: HDFS XAttrs (HDFS-2006)
    • Component/s: namenode
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Some javadoc improvements and minor comment fixups from HDFS-6299

      1. HDFS-6309.1.patch
        3 kB
        Charles Lamb
      2. HDFS-6309.2.patch
        5 kB
        Charles Lamb
      3. HDFS-6309.3.patch
        5 kB
        Charles Lamb

        Activity

        Hide
        Uma Maheswara Rao G added a comment -

        I have just committed this to branch!

        Show
        Uma Maheswara Rao G added a comment - I have just committed this to branch!
        Hide
        Uma Maheswara Rao G added a comment -

        Thanks a lot, Charles!
        Latest patch looks good to me. Will commit the patch to branch!.

        Show
        Uma Maheswara Rao G added a comment - Thanks a lot, Charles! Latest patch looks good to me. Will commit the patch to branch!.
        Hide
        Charles Lamb added a comment -

        ... as well you should. Sorry about that. A better patch has been attached.

        Show
        Charles Lamb added a comment - ... as well you should. Sorry about that. A better patch has been attached.
        Hide
        Charles Lamb added a comment -

        Sorry about that. The right patch is attached.

        Show
        Charles Lamb added a comment - Sorry about that. The right patch is attached.
        Hide
        Uma Maheswara Rao G added a comment -

        I will take back my +1 on this.

        Here your logic should be:

         if (result != null && result.isEmpty()) {
        

        to

         if (result != null && !result.isEmpty()) {
        
        Show
        Uma Maheswara Rao G added a comment - I will take back my +1 on this. Here your logic should be: if (result != null && result.isEmpty()) { to if (result != null && !result.isEmpty()) {
        Hide
        Uma Maheswara Rao G added a comment -

        Thanks a lot Charles for the patch.

        Patch looks good to me.
        +1

        I will commit the patch shortly to the branch!

        Show
        Uma Maheswara Rao G added a comment - Thanks a lot Charles for the patch. Patch looks good to me. +1 I will commit the patch shortly to the branch!
        Hide
        Charles Lamb added a comment -

        Fix typo in DFSClient.java

        Show
        Charles Lamb added a comment - Fix typo in DFSClient.java
        Hide
        Uma Maheswara Rao G added a comment -

        Patch looks good to me.
        Could you also consider the suggestion from Fengdong Yu at HDFS-6299?

        Another, try to use !result.isEmpty(), instead of 'result.size() > 0'

        Right, isEmpty() is efficient, and I will add this improvement.

        Show
        Uma Maheswara Rao G added a comment - Patch looks good to me. Could you also consider the suggestion from Fengdong Yu at HDFS-6299 ? Another, try to use !result.isEmpty(), instead of 'result.size() > 0' Right, isEmpty() is efficient, and I will add this improvement.
        Hide
        Charles Lamb added a comment -

        Here are some minor javadoc fixes and cleanups to XAttr.java.

        Show
        Charles Lamb added a comment - Here are some minor javadoc fixes and cleanups to XAttr.java.

          People

          • Assignee:
            Charles Lamb
            Reporter:
            Uma Maheswara Rao G
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development