Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Clean up some code style issues in HDFS-347.

      DomainSocket.java
      do not use AtomicInteger for status, add a new class
      rename fdRef(), fdUnref(boolean), jfds, jbuf, SND_BUF_SIZE, etc.
      do not override finalize().
      remove some dead code.

      1. HDFS-4476.002.patch
        25 kB
        Colin Patrick McCabe
      2. HDFS-4476.001.patch
        24 kB
        Colin Patrick McCabe

        Activity

        Hide
        Tsz Wo Nicholas Sze added a comment -

        Thanks for creating this JIRA. Please rename other short variable/method/function in other classes in the branch as well if possible.

        Show
        Tsz Wo Nicholas Sze added a comment - Thanks for creating this JIRA. Please rename other short variable/method/function in other classes in the branch as well if possible.
        Hide
        Aaron T. Myers added a comment -

        +1, the latest patch looks good to me. I'm going to commit this momentarily.

        Show
        Aaron T. Myers added a comment - +1, the latest patch looks good to me. I'm going to commit this momentarily.
        Hide
        Aaron T. Myers added a comment -

        I've just committed this to the HDFS-347 branch. Thanks a lot for the contribution, Colin.

        Show
        Aaron T. Myers added a comment - I've just committed this to the HDFS-347 branch. Thanks a lot for the contribution, Colin.

          People

          • Assignee:
            Colin Patrick McCabe
            Reporter:
            Colin Patrick McCabe
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development