YARN-4844. Addendum patch, revert few changes to public and evolving to avoid breaking downstream projects as much as possible.
This commit is contained in:
parent
159b249a3d
commit
7eb524ddee
|
@ -521,13 +521,14 @@ public class TypeConverter {
|
|||
application.getApplicationResourceUsageReport();
|
||||
if (resourceUsageReport != null) {
|
||||
jobStatus.setNeededMem(
|
||||
resourceUsageReport.getNeededResources().getMemorySize());
|
||||
(int)resourceUsageReport.getNeededResources().getMemorySize());
|
||||
jobStatus.setNumReservedSlots(
|
||||
resourceUsageReport.getNumReservedContainers());
|
||||
jobStatus.setNumUsedSlots(resourceUsageReport.getNumUsedContainers());
|
||||
jobStatus.setReservedMem(
|
||||
resourceUsageReport.getReservedResources().getMemorySize());
|
||||
jobStatus.setUsedMem(resourceUsageReport.getUsedResources().getMemorySize());
|
||||
(int)resourceUsageReport.getReservedResources().getMemorySize());
|
||||
jobStatus.setUsedMem(
|
||||
(int) resourceUsageReport.getUsedResources().getMemorySize());
|
||||
}
|
||||
return jobStatus;
|
||||
}
|
||||
|
|
|
@ -95,9 +95,9 @@ public class JobStatus implements Writable, Cloneable {
|
|||
private String trackingUrl ="";
|
||||
private int numUsedSlots;
|
||||
private int numReservedSlots;
|
||||
private long usedMem;
|
||||
private long reservedMem;
|
||||
private long neededMem;
|
||||
private int usedMem;
|
||||
private int reservedMem;
|
||||
private int neededMem;
|
||||
private boolean isUber;
|
||||
|
||||
/**
|
||||
|
@ -580,42 +580,42 @@ public class JobStatus implements Writable, Cloneable {
|
|||
/**
|
||||
* @return the used memory
|
||||
*/
|
||||
public long getUsedMem() {
|
||||
public int getUsedMem() {
|
||||
return usedMem;
|
||||
}
|
||||
|
||||
/**
|
||||
* @param m the used memory
|
||||
*/
|
||||
public void setUsedMem(long m) {
|
||||
public void setUsedMem(int m) {
|
||||
this.usedMem = m;
|
||||
}
|
||||
|
||||
/**
|
||||
* @return the reserved memory
|
||||
*/
|
||||
public long getReservedMem() {
|
||||
public int getReservedMem() {
|
||||
return reservedMem;
|
||||
}
|
||||
|
||||
/**
|
||||
* @param r the reserved memory
|
||||
*/
|
||||
public void setReservedMem(long r) {
|
||||
public void setReservedMem(int r) {
|
||||
this.reservedMem = r;
|
||||
}
|
||||
|
||||
/**
|
||||
* @return the needed memory
|
||||
*/
|
||||
public long getNeededMem() {
|
||||
public int getNeededMem() {
|
||||
return neededMem;
|
||||
}
|
||||
|
||||
/**
|
||||
* @param n the needed memory
|
||||
*/
|
||||
public void setNeededMem(long n) {
|
||||
public void setNeededMem(int n) {
|
||||
this.neededMem = n;
|
||||
}
|
||||
|
||||
|
|
|
@ -170,9 +170,9 @@ public class JobClientUnitTest {
|
|||
when(mockJobStatus.getPriority()).thenReturn(JobPriority.NORMAL);
|
||||
when(mockJobStatus.getNumUsedSlots()).thenReturn(1);
|
||||
when(mockJobStatus.getNumReservedSlots()).thenReturn(1);
|
||||
when(mockJobStatus.getUsedMem()).thenReturn(1024L);
|
||||
when(mockJobStatus.getReservedMem()).thenReturn(512L);
|
||||
when(mockJobStatus.getNeededMem()).thenReturn(2048L);
|
||||
when(mockJobStatus.getUsedMem()).thenReturn(1024);
|
||||
when(mockJobStatus.getReservedMem()).thenReturn(512);
|
||||
when(mockJobStatus.getNeededMem()).thenReturn(2048);
|
||||
when(mockJobStatus.getSchedulingInfo()).thenReturn("NA");
|
||||
|
||||
Job mockJob = mock(Job.class);
|
||||
|
|
|
@ -18,6 +18,7 @@
|
|||
|
||||
package org.apache.hadoop.yarn.api.records;
|
||||
|
||||
import org.apache.commons.lang.NotImplementedException;
|
||||
import org.apache.hadoop.classification.InterfaceAudience.Private;
|
||||
import org.apache.hadoop.classification.InterfaceAudience.Public;
|
||||
import org.apache.hadoop.classification.InterfaceStability.Unstable;
|
||||
|
@ -79,7 +80,9 @@ public abstract class Resource implements Comparable<Resource> {
|
|||
*/
|
||||
@Private
|
||||
@Unstable
|
||||
public abstract long getMemorySize();
|
||||
public long getMemorySize() {
|
||||
throw new NotImplementedException("getVirtualCoresSize is not implemented");
|
||||
}
|
||||
|
||||
/**
|
||||
* Set <em>memory</em> of the resource.
|
||||
|
@ -106,7 +109,9 @@ public abstract class Resource implements Comparable<Resource> {
|
|||
|
||||
@Public
|
||||
@Unstable
|
||||
public abstract long getVirtualCoresSize();
|
||||
public long getVirtualCoresSize() {
|
||||
throw new NotImplementedException("getVirtualCoresSize is not implemented");
|
||||
}
|
||||
|
||||
/**
|
||||
* Set <em>number of virtual cpu cores</em> of the resource.
|
||||
|
@ -120,7 +125,9 @@ public abstract class Resource implements Comparable<Resource> {
|
|||
*/
|
||||
@Public
|
||||
@Evolving
|
||||
public abstract void setVirtualCores(long vCores);
|
||||
public void setVirtualCores(long vCores) {
|
||||
throw new NotImplementedException("getVirtualCoresSize is not implemented");
|
||||
}
|
||||
|
||||
@Override
|
||||
public int hashCode() {
|
||||
|
|
Loading…
Reference in New Issue