From 0773c1aa0577e16e620b553f99690067d4e0e728 Mon Sep 17 00:00:00 2001 From: Eric Anderson Date: Thu, 13 Aug 2020 07:39:59 -0700 Subject: [PATCH] interop-testing: Remove comparison of proto field to null Proto fields aren't null. The getter will return an empty string instead of null. cl/326415191 --- .../testing/integration/GrpclbLongLivedAffinityTestClient.java | 3 --- 1 file changed, 3 deletions(-) diff --git a/interop-testing/src/main/java/io/grpc/testing/integration/GrpclbLongLivedAffinityTestClient.java b/interop-testing/src/main/java/io/grpc/testing/integration/GrpclbLongLivedAffinityTestClient.java index b98e4f67d3..efda4b9986 100644 --- a/interop-testing/src/main/java/io/grpc/testing/integration/GrpclbLongLivedAffinityTestClient.java +++ b/interop-testing/src/main/java/io/grpc/testing/integration/GrpclbLongLivedAffinityTestClient.java @@ -16,8 +16,6 @@ package io.grpc.testing.integration; -import static com.google.common.base.Preconditions.checkNotNull; - import com.google.protobuf.ByteString; import io.grpc.ManagedChannel; import io.grpc.StatusRuntimeException; @@ -175,7 +173,6 @@ public final class GrpclbLongLivedAffinityTestClient { blockingStub.withDeadlineAfter(1, TimeUnit.MINUTES).unaryCall(request); logger.info("Received response"); String serverId = response.getServerId(); - checkNotNull(serverId, "serverId is null"); if (lastServerId != null && !lastServerId.equals(serverId)) { String msg = "Expected serverId " + lastServerId + ", but got " + serverId; logger.warning(msg + ". affinityBreakageBudget=" + affinityBreakageBudget);