diff options
author | Harald Musum <musum@yahoo-inc.com> | 2016-07-05 08:29:44 +0200 |
---|---|---|
committer | Harald Musum <musum@yahoo-inc.com> | 2016-07-05 08:29:44 +0200 |
commit | c706353aed19380c119b9f5492472c09f4904eec (patch) | |
tree | fe4492545df9250a2e5c7cd258d4f48bee85d40a /orchestrator | |
parent | ac0d36d85cec4f961d5b9008c2d7fc2e5e943160 (diff) |
Rename variables after code review feedback
Diffstat (limited to 'orchestrator')
-rw-r--r-- | orchestrator/src/main/java/com/yahoo/vespa/orchestrator/resources/HostSuspensionResource.java | 10 |
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 { |