[ https://issues.apache.org/jira/browse/MAPREDUCE-7146?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jason Lowe moved YARN-8832 to MAPREDUCE-7146: --------------------------------------------- Affects Version/s: (was: 3.2.0) 3.2.0 Component/s: (was: applications) mr-am Key: MAPREDUCE-7146 (was: YARN-8832) Project: Hadoop Map/Reduce (was: Hadoop YARN) > Review of RMCommunicator Class > ------------------------------ > > Key: MAPREDUCE-7146 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-7146 > Project: Hadoop Map/Reduce > Issue Type: Improvement > Components: mr-am > Affects Versions: 3.2.0 > Reporter: BELUGA BEHR > Assignee: BELUGA BEHR > Priority: Minor > Attachments: YARN-88321.patch > > > 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. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org