Szilard Nemeth created YARN-9100:
------------------------------------
Summary: Add tests for GpuResourceAllocator and do minor code
cleanup
Key: YARN-9100
URL: https://issues.apache.org/jira/browse/YARN-9100
Project: Hadoop YARN
Issue Type: Improvement
Reporter: Szilard Nemeth
Assignee: Szilard Nemeth
Add tests for GpuResourceAllocator and do minor code cleanup
- Improved log and exception messages
- Added some new debug logs
- Some methods are named like *Copy, these are returning copies of internal
data structures. The word "copy" is just a noise in their name, so they have
been renamed. Additionally, the copied data structures modified to be immutable.
- The waiting loop in method assignGpus were decoupled into a new class,
RetryCommand.
Some more words about the new class RetryCommand:
There are some similar waiting loops in the code in: AMRMClient,
AMRMClientAsync and even in GenericTestUtils (see waitFor method). RetryCommand
could be a future replacement of these duplicated code, as it gives a solution
to this waiting loop problem in a generic way.
The only downside of the usage of RetryCommand in GpuResourceAllocator
(startGpuAssignmentLoop) is the ugly exception handling part, but that's solely
because how Java deals with checked exceptions vs. lambdas. If there's a
cleaner way to solve the exception handling, I'm open for any suggestions.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-dev-h...@hadoop.apache.org
