summaryrefslogtreecommitdiffstats
path: root/orchestrator
diff options
context:
space:
mode:
authorHarald Musum <musum@yahoo-inc.com>2016-07-05 08:29:44 +0200
committerHarald Musum <musum@yahoo-inc.com>2016-07-05 08:29:44 +0200
commitc706353aed19380c119b9f5492472c09f4904eec (patch)
treefe4492545df9250a2e5c7cd258d4f48bee85d40a /orchestrator
parentac0d36d85cec4f961d5b9008c2d7fc2e5e943160 (diff)
Rename variables after code review feedback
Diffstat (limited to 'orchestrator')
-rw-r--r--orchestrator/src/main/java/com/yahoo/vespa/orchestrator/resources/HostSuspensionResource.java10
1 files changed, 5 insertions, 5 deletions
diff --git a/orchestrator/src/main/java/com/yahoo/vespa/orchestrator/resources/HostSuspensionResource.java b/orchestrator/src/main/java/com/yahoo/vespa/orchestrator/resources/HostSuspensionResource.java
index 9719c602a22..c342e378532 100644
--- a/orchestrator/src/main/java/com/yahoo/vespa/orchestrator/resources/HostSuspensionResource.java
+++ b/orchestrator/src/main/java/com/yahoo/vespa/orchestrator/resources/HostSuspensionResource.java
@@ -34,20 +34,20 @@ public class HostSuspensionResource implements HostSuspensionApi {
@Override
public BatchOperationResult suspendAll(BatchHostSuspendRequest request) throws WebApplicationException {
- final String ph = request.getParentHostname();
- if (ph == null || ph.isEmpty()) {
+ final String parentHostnameString = request.getParentHostname();
+ if (parentHostnameString == null || parentHostnameString.isEmpty()) {
String message = "parentHostname missing or empty in request: " + request;
log.log(LogLevel.DEBUG, message);
throw createWebApplicationException(message, Response.Status.BAD_REQUEST);
}
- final List<String> h = request.getHostnames();
- if (h == null || h.isEmpty()) {
+ final List<String> hostnamesAsStrings = request.getHostnames();
+ if (hostnamesAsStrings == null || hostnamesAsStrings.isEmpty()) {
String message = "hostnames missing in request: " + request;
log.log(LogLevel.DEBUG, message);
throw createWebApplicationException(message, Response.Status.BAD_REQUEST);
}
- HostName parentHostname = new HostName(ph);
+ HostName parentHostname = new HostName(parentHostnameString);
List<HostName> hostNames = request.getHostnames().stream().map(HostName::new).collect(Collectors.toList());
try {