Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
1.1.0
-
None
-
Reviewed
Description
In MAPREDUCE-3789, the fix had unfortunately only covered the first time assignment scenario, and the test had not really caught the mistake of using the condition of looking at available TT slots (instead of looking for how many slots a job's task demands).
We should change the condition of reservation in such a manner:
if ((getPendingTasks(j) != 0 &&
!hasSufficientReservedTaskTrackers(j)) &&
- (taskTracker.getAvailableSlots(type) !=
+ !(j.getNumSlotsPerTask(type) >
getTTMaxSlotsForType(taskTrackerStatus, type))) {
I had not realized during the earlier ticket that j.getNumSlotsPerTask(type) did exist.
Attachments
Attachments
Issue Links
- relates to
-
MAPREDUCE-516 Fix the 'cluster drain' problem in the Capacity Scheduler wrt High RAM Jobs
- Closed