Skip to content

Commit

Permalink
YARN-5203.Return ResourceRequest JAXB object in ResourceManager Clust…
Browse files Browse the repository at this point in the history
…er Applications REST API. Contributed by Ellen Hui.
  • Loading branch information
subru committed Jul 28, 2016
1 parent a1890c3 commit 4e756d7
Show file tree
Hide file tree
Showing 8 changed files with 330 additions and 19 deletions.
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@
import org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttempt; import org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttempt;
import org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptMetrics; import org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptMetrics;
import org.apache.hadoop.yarn.server.resourcemanager.webapp.dao.AppInfo; import org.apache.hadoop.yarn.server.resourcemanager.webapp.dao.AppInfo;
import org.apache.hadoop.yarn.server.resourcemanager.webapp.dao.ResourceRequestInfo;
import org.apache.hadoop.yarn.server.webapp.AppAttemptBlock; import org.apache.hadoop.yarn.server.webapp.AppAttemptBlock;
import org.apache.hadoop.yarn.server.webapp.dao.AppAttemptInfo; import org.apache.hadoop.yarn.server.webapp.dao.AppAttemptInfo;
import org.apache.hadoop.yarn.util.Times; import org.apache.hadoop.yarn.util.Times;
Expand Down Expand Up @@ -71,7 +72,7 @@ private void createResourceRequestsTable(Block html) {
.get(this.appAttemptId.getApplicationId()), true, .get(this.appAttemptId.getApplicationId()), true,
WebAppUtils.getHttpSchemePrefix(conf)); WebAppUtils.getHttpSchemePrefix(conf));


List<ResourceRequest> resourceRequests = app.getResourceRequests(); List<ResourceRequestInfo> resourceRequests = app.getResourceRequests();
if (resourceRequests == null || resourceRequests.isEmpty()) { if (resourceRequests == null || resourceRequests.isEmpty()) {
return; return;
} }
Expand All @@ -88,7 +89,7 @@ private void createResourceRequestsTable(Block html) {
.th(".labelexpression", "NodeLabelExpression")._()._().tbody(); .th(".labelexpression", "NodeLabelExpression")._()._().tbody();


StringBuilder resourceRequestTableData = new StringBuilder("[\n"); StringBuilder resourceRequestTableData = new StringBuilder("[\n");
for (ResourceRequest resourceRequest : resourceRequests) { for (ResourceRequestInfo resourceRequest : resourceRequests) {
if (resourceRequest.getNumContainers() == 0) { if (resourceRequest.getNumContainers() == 0) {
continue; continue;
} }
Expand Down Expand Up @@ -118,19 +119,19 @@ private void createResourceRequestsTable(Block html) {
div._(); div._();
} }


private Resource getTotalResource(List<ResourceRequest> requests) { private Resource getTotalResource(List<ResourceRequestInfo> requests) {
Resource totalResource = Resource.newInstance(0, 0); Resource totalResource = Resource.newInstance(0, 0);
if (requests == null) { if (requests == null) {
return totalResource; return totalResource;
} }
for (ResourceRequest request : requests) { for (ResourceRequestInfo request : requests) {
if (request.getNumContainers() == 0) { if (request.getNumContainers() == 0) {
continue; continue;
} }
if (request.getResourceName().equals(ResourceRequest.ANY)) { if (request.getResourceName().equals(ResourceRequest.ANY)) {
Resources.addTo( Resources.addTo(
totalResource, totalResource,
Resources.multiply(request.getCapability(), Resources.multiply(request.getCapability().getResource(),
request.getNumContainers())); request.getNumContainers()));
} }
} }
Expand Down
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -17,10 +17,12 @@
*/ */
package org.apache.hadoop.yarn.server.resourcemanager.webapp.dao; package org.apache.hadoop.yarn.server.resourcemanager.webapp.dao;


import java.util.ArrayList;
import java.util.List; import java.util.List;


import javax.xml.bind.annotation.XmlAccessType; import javax.xml.bind.annotation.XmlAccessType;
import javax.xml.bind.annotation.XmlAccessorType; import javax.xml.bind.annotation.XmlAccessorType;
import javax.xml.bind.annotation.XmlElement;
import javax.xml.bind.annotation.XmlRootElement; import javax.xml.bind.annotation.XmlRootElement;
import javax.xml.bind.annotation.XmlTransient; import javax.xml.bind.annotation.XmlTransient;


Expand Down Expand Up @@ -97,7 +99,10 @@ public class AppInfo {
protected int numNonAMContainerPreempted; protected int numNonAMContainerPreempted;
protected int numAMContainerPreempted; protected int numAMContainerPreempted;


protected List<ResourceRequest> resourceRequests; // list of resource requests
@XmlElement(name = "resourceRequests")
private List<ResourceRequestInfo> resourceRequests =
new ArrayList<ResourceRequestInfo>();


protected LogAggregationStatus logAggregationStatus; protected LogAggregationStatus logAggregationStatus;
protected boolean unmanagedApplication; protected boolean unmanagedApplication;
Expand Down Expand Up @@ -182,8 +187,16 @@ public AppInfo(ResourceManager rm, RMApp app, Boolean hasAccess,
queueUsagePercentage = resourceReport.getQueueUsagePercentage(); queueUsagePercentage = resourceReport.getQueueUsagePercentage();
clusterUsagePercentage = resourceReport.getClusterUsagePercentage(); clusterUsagePercentage = resourceReport.getClusterUsagePercentage();
} }
resourceRequests = rm.getRMContext().getScheduler()
List<ResourceRequest> resourceRequestsRaw = rm.getRMContext()
.getScheduler()
.getPendingResourceRequestsForAttempt(attempt.getAppAttemptId()); .getPendingResourceRequestsForAttempt(attempt.getAppAttemptId());

if (resourceRequestsRaw != null) {
for (ResourceRequest req : resourceRequestsRaw) {
resourceRequests.add(new ResourceRequestInfo(req));
}
}
} }
} }


Expand Down Expand Up @@ -352,7 +365,7 @@ public long getVcoreSeconds() {
return vcoreSeconds; return vcoreSeconds;
} }


public List<ResourceRequest> getResourceRequests() { public List<ResourceRequestInfo> getResourceRequests() {
return this.resourceRequests; return this.resourceRequests;
} }


Expand Down
Original file line number Original file line Diff line number Diff line change
@@ -0,0 +1,63 @@
/**
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

package org.apache.hadoop.yarn.server.resourcemanager.webapp.dao;

import javax.xml.bind.annotation.XmlAccessType;
import javax.xml.bind.annotation.XmlAccessorType;
import javax.xml.bind.annotation.XmlElement;
import javax.xml.bind.annotation.XmlRootElement;

import org.apache.hadoop.yarn.api.records.ExecutionType;
import org.apache.hadoop.yarn.api.records.ExecutionTypeRequest;

/**
* Simple class representing an execution type request.
*/
@XmlRootElement(name = "ExecutionTypeRequest")
@XmlAccessorType(XmlAccessType.FIELD)
public class ExecutionTypeRequestInfo {
@XmlElement(name = "executionType")
private String executionType;
@XmlElement(name = "enforceExecutionType")
private boolean enforceExecutionType;

public ExecutionTypeRequestInfo() {
}

public ExecutionTypeRequestInfo(ExecutionTypeRequest executionTypeRequest) {
executionType = executionTypeRequest.getExecutionType().name();
enforceExecutionType = executionTypeRequest.getEnforceExecutionType();
}

public ExecutionType getExecutionType() {
return ExecutionType.valueOf(executionType);
}

public void setExecutionType(ExecutionType executionType) {
this.executionType = executionType.name();
}

public boolean getEnforceExecutionType() {
return enforceExecutionType;
}

public void setEnforceExecutionType(boolean enforceExecutionType) {
this.enforceExecutionType = enforceExecutionType;
}
}
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -58,4 +58,8 @@ public void setMemory(int memory) {
public void setvCores(int vCores) { public void setvCores(int vCores) {
this.vCores = vCores; this.vCores = vCores;
} }

public Resource getResource() {
return Resource.newInstance(memory, vCores);
}
} }
Original file line number Original file line Diff line number Diff line change
@@ -0,0 +1,131 @@
/**
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

package org.apache.hadoop.yarn.server.resourcemanager.webapp.dao;

import org.apache.hadoop.yarn.api.records.ExecutionTypeRequest;
import org.apache.hadoop.yarn.api.records.Priority;
import org.apache.hadoop.yarn.api.records.ResourceRequest;

import javax.xml.bind.annotation.XmlAccessType;
import javax.xml.bind.annotation.XmlAccessorType;
import javax.xml.bind.annotation.XmlElement;
import javax.xml.bind.annotation.XmlRootElement;

/**
* Simple class representing a resource request.
*/
@XmlRootElement(name = "resourceRequests")
@XmlAccessorType(XmlAccessType.FIELD)
public class ResourceRequestInfo {

@XmlElement(name = "priority")
private int priority;
@XmlElement(name = "resourceName")
private String resourceName;
@XmlElement(name = "capability")
private ResourceInfo capability;
@XmlElement(name = "numContainers")
private int numContainers;
@XmlElement(name = "relaxLocality")
private boolean relaxLocality;
@XmlElement(name = "nodeLabelExpression")
private String nodeLabelExpression;

@XmlElement(name = "executionTypeRequest")
private ExecutionTypeRequestInfo executionTypeRequest;

@XmlElement(name = "executionType")
private String executionType;
@XmlElement(name = "enforceExecutionType")
private boolean enforceExecutionType;

public ResourceRequestInfo() {
}

public ResourceRequestInfo(ResourceRequest request) {
priority = request.getPriority().getPriority();
resourceName = request.getResourceName();
capability = new ResourceInfo(request.getCapability());
numContainers = request.getNumContainers();
relaxLocality = request.getRelaxLocality();
nodeLabelExpression = request.getNodeLabelExpression();
if (request.getExecutionTypeRequest() != null) {
executionTypeRequest =
new ExecutionTypeRequestInfo(request.getExecutionTypeRequest());
}
}

public Priority getPriority() {
return Priority.newInstance(priority);
}

public void setPriority(Priority priority) {
this.priority = priority.getPriority();
}

public String getResourceName() {
return resourceName;
}

public void setResourceName(String resourceName) {
this.resourceName = resourceName;
}

public ResourceInfo getCapability() {
return capability;
}

public void setCapability(ResourceInfo capability) {
this.capability = capability;
}

public int getNumContainers() {
return numContainers;
}

public void setNumContainers(int numContainers) {
this.numContainers = numContainers;
}

public boolean getRelaxLocality() {
return relaxLocality;
}

public void setRelaxLocality(boolean relaxLocality) {
this.relaxLocality = relaxLocality;
}

public String getNodeLabelExpression() {
return nodeLabelExpression;
}

public void setNodeLabelExpression(String nodeLabelExpression) {
this.nodeLabelExpression = nodeLabelExpression;
}

public void setExecutionTypeRequest(
ExecutionTypeRequest executionTypeRequest) {
this.executionTypeRequest =
new ExecutionTypeRequestInfo(executionTypeRequest);
}

public ExecutionTypeRequestInfo getExecutionTypeRequest() {
return executionTypeRequest;
}
}
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
import org.apache.hadoop.yarn.api.records.Priority; import org.apache.hadoop.yarn.api.records.Priority;
import org.apache.hadoop.yarn.api.records.Resource; import org.apache.hadoop.yarn.api.records.Resource;
import org.apache.hadoop.yarn.api.records.ResourceRequest; import org.apache.hadoop.yarn.api.records.ResourceRequest;
import org.apache.hadoop.yarn.api.records.ExecutionTypeRequest;
import org.apache.hadoop.yarn.security.AMRMTokenIdentifier; import org.apache.hadoop.yarn.security.AMRMTokenIdentifier;
import org.apache.hadoop.yarn.server.resourcemanager.rmapp.RMApp; import org.apache.hadoop.yarn.server.resourcemanager.rmapp.RMApp;
import org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttempt; import org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttempt;
Expand Down Expand Up @@ -197,8 +198,15 @@ public ResourceRequest createResourceReq(String resource, int memory, int priori
return createResourceReq(resource, memory, priority, containers, null); return createResourceReq(resource, memory, priority, containers, null);
} }


public ResourceRequest createResourceReq(String resource, int memory, int priority, public ResourceRequest createResourceReq(String resource, int memory,
int containers, String labelExpression) throws Exception { int priority, int containers, String labelExpression) throws Exception {
return createResourceReq(resource, memory, priority, containers,
labelExpression, ExecutionTypeRequest.newInstance());
}

public ResourceRequest createResourceReq(String resource, int memory,
int priority, int containers, String labelExpression,
ExecutionTypeRequest executionTypeRequest) throws Exception {
ResourceRequest req = Records.newRecord(ResourceRequest.class); ResourceRequest req = Records.newRecord(ResourceRequest.class);
req.setResourceName(resource); req.setResourceName(resource);
req.setNumContainers(containers); req.setNumContainers(containers);
Expand All @@ -209,9 +217,11 @@ public ResourceRequest createResourceReq(String resource, int memory, int priori
capability.setMemorySize(memory); capability.setMemorySize(memory);
req.setCapability(capability); req.setCapability(capability);
if (labelExpression != null) { if (labelExpression != null) {
req.setNodeLabelExpression(labelExpression); req.setNodeLabelExpression(labelExpression);
} }
req.setExecutionTypeRequest(executionTypeRequest);
return req; return req;

} }


public AllocateResponse allocate( public AllocateResponse allocate(
Expand Down
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -613,12 +613,12 @@ public RMApp submitApp(Resource capability, String name, String user,
sub.setLogAggregationContext(logAggregationContext); sub.setLogAggregationContext(logAggregationContext);
} }
sub.setCancelTokensWhenComplete(cancelTokensWhenComplete); sub.setCancelTokensWhenComplete(cancelTokensWhenComplete);
ResourceRequest amResourceRequest = ResourceRequest.newInstance(
Priority.newInstance(0), ResourceRequest.ANY, capability, 1);
if (amLabel != null && !amLabel.isEmpty()) { if (amLabel != null && !amLabel.isEmpty()) {
ResourceRequest amResourceRequest = ResourceRequest.newInstance(
Priority.newInstance(0), ResourceRequest.ANY, capability, 1);
amResourceRequest.setNodeLabelExpression(amLabel.trim()); amResourceRequest.setNodeLabelExpression(amLabel.trim());
sub.setAMContainerResourceRequest(amResourceRequest);
} }
sub.setAMContainerResourceRequest(amResourceRequest);
req.setApplicationSubmissionContext(sub); req.setApplicationSubmissionContext(sub);
UserGroupInformation fakeUser = UserGroupInformation fakeUser =
UserGroupInformation.createUserForTesting(user, new String[] {"someGroup"}); UserGroupInformation.createUserForTesting(user, new String[] {"someGroup"});
Expand Down
Loading

0 comments on commit 4e756d7

Please sign in to comment.