Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-5963

unnecessary exception catch in NNBench

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

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.21.0
    • Component/s: test
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      NNBench.createControlFiles - catches Exception (even though only IOException is thrown) and then throws a new IOException with message from original. This seems unnecessary and lossfull, because we loose stack info form the original exception.

      Suggestion - remove the catch part of try block.

        Attachments

        1. Hadoop-5963.patch
          0.7 kB
          Boris Shkolnik

          Activity

            People

            • Assignee:
              boryas Boris Shkolnik
              Reporter:
              boryas Boris Shkolnik

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment