Skip to content

Commit 14f4ce0

Browse files
committed
Use String utility
Signed-off-by: Rishab Nahata <rnnahata@amazon.com>
1 parent fc6dd88 commit 14f4ce0

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

server/src/main/java/org/opensearch/action/admin/cluster/decommission/awareness/put/DecommissionRequest.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
import org.opensearch.action.ActionRequestValidationException;
1212
import org.opensearch.action.support.clustermanager.ClusterManagerNodeRequest;
1313
import org.opensearch.cluster.decommission.DecommissionAttribute;
14+
import org.opensearch.common.Strings;
1415
import org.opensearch.common.io.stream.StreamInput;
1516
import org.opensearch.common.io.stream.StreamOutput;
1617

@@ -67,10 +68,10 @@ public DecommissionAttribute getDecommissionAttribute() {
6768
@Override
6869
public ActionRequestValidationException validate() {
6970
ActionRequestValidationException validationException = null;
70-
if (decommissionAttribute.attributeName() == null || decommissionAttribute.attributeName().isEmpty()) {
71+
if (decommissionAttribute.attributeName() == null || Strings.isEmpty(decommissionAttribute.attributeName())) {
7172
validationException = addValidationError("attribute name is missing", validationException);
7273
}
73-
if (decommissionAttribute.attributeValue() == null || decommissionAttribute.attributeValue().isEmpty()) {
74+
if (decommissionAttribute.attributeValue() == null || Strings.isEmpty(decommissionAttribute.attributeValue())) {
7475
validationException = addValidationError("attribute value is missing", validationException);
7576
}
7677
return validationException;

0 commit comments

Comments
 (0)