From 5322c951ffaac38bc89c93ab510f312412fe2c53 Mon Sep 17 00:00:00 2001 From: skuruppu Date: Thu, 22 Oct 2020 20:23:33 +1100 Subject: [PATCH] test: remove explicit CreateInstance timeout (#524) Use the default settings for polling long-running operations instead. This change is made to prevent test flakiness from having too short of a timeout. --- .../test/java/com/google/cloud/spanner/IntegrationTestEnv.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/google-cloud-spanner/src/test/java/com/google/cloud/spanner/IntegrationTestEnv.java b/google-cloud-spanner/src/test/java/com/google/cloud/spanner/IntegrationTestEnv.java index b67f970273..687bb44ba1 100644 --- a/google-cloud-spanner/src/test/java/com/google/cloud/spanner/IntegrationTestEnv.java +++ b/google-cloud-spanner/src/test/java/com/google/cloud/spanner/IntegrationTestEnv.java @@ -24,7 +24,6 @@ import com.google.spanner.admin.instance.v1.CreateInstanceMetadata; import io.grpc.Status; import java.util.Random; -import java.util.concurrent.TimeUnit; import java.util.logging.Level; import java.util.logging.Logger; import org.junit.rules.ExternalResource; @@ -126,7 +125,7 @@ private void initializeInstance(InstanceId instanceId) { instanceAdminClient.createInstance(instance); Instance createdInstance; try { - createdInstance = op.get(30000L, TimeUnit.MILLISECONDS); + createdInstance = op.get(); } catch (Exception e) { boolean cancelled = false; try {