Details

    • Type: Improvement
    • Status: Patch Available
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.2.0
    • Fix Version/s: None
    • Component/s: mr-am
    • Labels:
      None
    • Flags:
      Patch

      Description

      Various improvements to the RMCommunicator class.

       

      • Use SLF4J parameterized logging
      • Use switch statement instead of if-else statements
      • Remove anti-pattern of "log and throw" (just throw)
      • Use a flag to stop thread instead of an interrupt (it may be interrupting the heartbeat code and not the thread loop)
      • The main thread loops performs loops on the heartbeat callback queue until the queue is empty.  It's technically possible that other threads could constantly put new callbacks into the queue and therefore the main thread never progresses past the callbacks.  Put a cap on the number of callbacks that will be processed in any iteration.

        Attachments

        1. YARN-88321.patch
          11 kB
          David Mollitor

          Activity

            People

            • Assignee:
              belugabehr David Mollitor
              Reporter:
              belugabehr David Mollitor
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated: