[YARN-9108] fix FederationIntercepter merge home and secondary allocate response typo

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Resolved
• Affects Version/s: 3.3.0
• Fix Version/s: 2.10.0, 3.3.0
• Component/s: federation
• Assignee: Abhishek Modi
• Reporter: Morty Zhong
• Created: 11/Dec/18 09:49
• Updated: 23/Dec/18 04:02
• Resolved: 22/Dec/18 20:49

Description

method 'mergeAllocateResponse' in class FederationIntercepter.java line 1315

the left variable `par2` should be `par1`

if (par1 != null && par2 != null) {
  par1.getResourceRequest().addAll(par2.getResourceRequest());
  par2.getContainers().addAll(par2.getContainers());
}

should be

if (par1 != null && par2 != null) {
  par1.getResourceRequest().addAll(par2.getResourceRequest());
  par1.getContainers().addAll(par2.getContainers());//edited line
}