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

increase readability and reliability of checkstyle, shellcheck, and whitespace reports

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

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 2.8.0, 3.0.0-alpha1
    • None
    • None

    Description

      HADOOP-11746 supports listing of the lines which has trailing white spaces but doesn't inform patch line number. Without this report output will not be of much help as in most cases it reports blank lines. Also for the first timers it would be difficult to understand the output check style script hence adding an header

      Attachments

        1. HADOOP-11866.20150422-1.patch
          0.6 kB
          Naganarasimha G R
        2. HADOOP-11866.20150423-1.patch
          2 kB
          Naganarasimha G R
        3. HADOOP-11866.20150427-1.patch
          8 kB
          Naganarasimha G R
        4. HADOOP-11866-05.patch
          13 kB
          Allen Wittenauer
        5. HADOOP-11866-06.patch
          16 kB
          Allen Wittenauer
        6. HADOOP-11866-07.patch
          17 kB
          Allen Wittenauer
        7. HADOOP-11866-08.patch
          17 kB
          Allen Wittenauer
        8. HADOOP-11866-checkstyle.patch
          7 kB
          Allen Wittenauer

        Issue Links

        depends upon

        Test - A new unit, integration or system test. HADOOP-11746 rewrite test-patch.sh

        • Major - Major loss of function.
        • Resolved
        duplicates

        Bug - A problem which impairs or prevents the functions of the product. HADOOP-11869 Suppress ParameterNumber checkstyle violations for overridden methods

        • Major - Major loss of function.
        • Resolved
        is related to

        Improvement - An improvement or enhancement to an existing feature or task. HADOOP-11778 Update checkstyle

        • Major - Major loss of function.
        • Resolved

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

          People

            aw Allen Wittenauer
            Naganarasimha Naganarasimha G R
            Votes:
            0 Vote for this issue
            Watchers:
            11 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment