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

Log improvements in NodeStatusUpdaterImpl

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

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 3.2.2, 3.4.0, 3.3.1, 3.2.3
    • nodemanager
    • None

    Description

      I think in below places log should be printed only if list size is not zero.

      if (LOG.isDebugEnabled()) {
                LOG.debug("The cache log aggregation status size:"
                    + logAggregationReports.size());
       }
      
      LOG.info("Sending out " + containerStatuses.size()
            + " NM container statuses: " + containerStatuses);
      
      if (LOG.isDebugEnabled()) {
            LOG.debug("Sending out " + containerStatuses.size()
                + " container statuses: " + containerStatuses);
      }
      

      Attachments

        1. YARN-10430.001.patch
          3 kB
          Bilwa S T

        Activity

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

          People

            BilwaST Bilwa S T
            BilwaST Bilwa S T
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment