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

Remove logging exception stack trace when one of the datanode targets to read from is not reachable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.20.203.1
    • Fix Version/s: 0.23.0
    • Component/s: hdfs-client
    • Labels:
      None

      Description

      When client fails to connect to one of the datanodes from the list of block locations returned, exception stack trace is printed in the client log. This is an expected failure scenario that is handled at the client, by going to the next location. Printing entire stack trace is unnecessary and just printing the exception message should be sufficient.

      1. HDFS-1906.patch
        1 kB
        Suresh Srinivas
      2. HDFS-1906.2.patch
        1 kB
        Suresh Srinivas
      3. HDFS-1906.rel205.patch
        2 kB
        Suresh Srinivas

        Activity

        Hide
        hudson Hudson added a comment -

        Integrated in Hadoop-Hdfs-trunk #673 (See https://builds.apache.org/hudson/job/Hadoop-Hdfs-trunk/673/)

        Show
        hudson Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #673 (See https://builds.apache.org/hudson/job/Hadoop-Hdfs-trunk/673/ )
        Hide
        szetszwo Tsz Wo Nicholas Sze added a comment -

        > Patch for 0.20.205
        +1

        Show
        szetszwo Tsz Wo Nicholas Sze added a comment - > Patch for 0.20.205 +1
        Hide
        sureshms Suresh Srinivas added a comment -

        Patch for 0.20.205

        Show
        sureshms Suresh Srinivas added a comment - Patch for 0.20.205
        Hide
        hudson Hudson added a comment -

        Integrated in Hadoop-Hdfs-trunk-Commit #637 (See https://builds.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/637/)

        Show
        hudson Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #637 (See https://builds.apache.org/hudson/job/Hadoop-Hdfs-trunk-Commit/637/ )
        Hide
        sureshms Suresh Srinivas added a comment -

        I committed the patch.

        Show
        sureshms Suresh Srinivas added a comment - I committed the patch.
        Hide
        sureshms Suresh Srinivas added a comment -

        No tests are added in this patch, because this is a simple log change. I am also not planning to run hudson tests.

        Show
        sureshms Suresh Srinivas added a comment - No tests are added in this patch, because this is a simple log change. I am also not planning to run hudson tests.
        Hide
        szetszwo Tsz Wo Nicholas Sze added a comment -

        +1 patch looks good.

        Show
        szetszwo Tsz Wo Nicholas Sze added a comment - +1 patch looks good.
        Hide
        sureshms Suresh Srinivas added a comment -

        Updated patch prints exception stack trace in debug logs.

        Show
        sureshms Suresh Srinivas added a comment - Updated patch prints exception stack trace in debug logs.
        Hide
        szetszwo Tsz Wo Nicholas Sze added a comment -

        I think the stack trace is useful for debug. How about printing it if debug is enabled?

        Show
        szetszwo Tsz Wo Nicholas Sze added a comment - I think the stack trace is useful for debug. How about printing it if debug is enabled?
        Hide
        eli Eli Collins added a comment -

        +1 lgtm

        Show
        eli Eli Collins added a comment - +1 lgtm

          People

          • Assignee:
            sureshms Suresh Srinivas
            Reporter:
            sureshms Suresh Srinivas
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development