Uploaded image for project: 'Hadoop HDFS'
  1. Hadoop HDFS
  2. HDFS-9700

DFSClient and DFSOutputStream should set TCP_NODELAY on sockets for DataTransferProtocol

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.7.1, 2.6.3
    • 2.8.0, 3.0.0-alpha1
    • hdfs-client
    • None

    Description

      In DFSClient.connectToDN() and DFSOutputStream.createSocketForPipeline(), we never call setTcpNoDelay() on the constructed socket before sending. In both cases, we should respect the value of ipc.client.tcpnodelay in the configuration.

      While this applies whether security is enabled or not, it seems to have a bigger impact on latency when security is enabled.

      Attachments

        1. HDFS-9700_branch-2.7.patch
          3 kB
          Gary Helmling
        2. HDFS-9700_branch-2.7-v2.patch
          4 kB
          Gary Helmling
        3. HDFS-9700.002.patch
          5 kB
          Gary Helmling
        4. HDFS-9700.003.patch
          5 kB
          Gary Helmling
        5. HDFS-9700.004.patch
          5 kB
          Gary Helmling
        6. HDFS-9700-branch-2.7.002.patch
          4 kB
          Gary Helmling
        7. HDFS-9700-branch-2.7.003.patch
          4 kB
          Gary Helmling
        8. HDFS-9700-v1.patch
          3 kB
          Gary Helmling
        9. HDFS-9700-v2.patch
          5 kB
          Gary Helmling

        Issue Links

        is related to

        Improvement - An improvement or enhancement to an existing feature or task. HADOOP-8069 Enable TCP_NODELAY by default for IPC

        • Major - Major loss of function.
        • Closed

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            ghelmling Gary Helmling
            ghelmling Gary Helmling
            Votes:
            0 Vote for this issue
            Watchers:
            10 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment