Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-5776

Checkstyle: MonitoringThread.Run method length is too long

Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Reviewed

    Description

      YARN-5725 had a check style violation that should be resolved by refactoring the function

      Details:
      ContainersMonitorImpl.java:395 MonitioringThread.Run @Override:5: Method length is 233 lines (max allowed is 150).

      Attachments

        Issue Links

        causes

        Bug - A problem which impairs or prevents the functions of the product. YARN-9584 Should put initializeProcessTrees method call before get pid

        • Critical - Crashes, loss of data, severe memory leak.
        • Resolved
        is duplicated by

        Improvement - An improvement or enhancement to an existing feature or task. YARN-6581 Function ContainersMonitorImpl.MonitoringThread#run() is too long

        • Major - Major loss of function.
        • Resolved
        relates to

        Bug - A problem which impairs or prevents the functions of the product. YARN-5725 Test uncaught exception in TestContainersMonitorResourceChange.testContainersResourceChange when setting IP and host

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Resolved

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

          People

            miklos.szegedi@cloudera.com Miklos Szegedi Assign to me
            miklos.szegedi@cloudera.com Miklos Szegedi
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment