From a3c89f53a5a159be955dc392d74836185a22d5cd Mon Sep 17 00:00:00 2001 From: Mira Leung Date: Tue, 17 Nov 2020 13:29:40 -0800 Subject: [PATCH 1/4] fix: Update LRO initial_retry_delayo o 5s (discussion in doc) --- .../gapic/composer/RetrySettingsComposer.java | 2 +- .../gapic/composer/RetrySettingsComposerTest.java | 2 +- .../gapic/composer/goldens/EchoStubSettings.golden | 2 +- .../goldens/redis/CloudRedisStubSettings.java | 14 +++++++------- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/main/java/com/google/api/generator/gapic/composer/RetrySettingsComposer.java b/src/main/java/com/google/api/generator/gapic/composer/RetrySettingsComposer.java index 7f599c9d8b..a1f9acd2d2 100644 --- a/src/main/java/com/google/api/generator/gapic/composer/RetrySettingsComposer.java +++ b/src/main/java/com/google/api/generator/gapic/composer/RetrySettingsComposer.java @@ -60,7 +60,7 @@ public class RetrySettingsComposer { TypeNode.withReference(ConcreteReference.withClazz(StatusCode.Code.class)); // Default values for LongRunningConfig fields. - private static final long LRO_DEFAULT_INITIAL_POLL_DELAY_MILLIS = 20000; + private static final long LRO_DEFAULT_INITIAL_POLL_DELAY_MILLIS = 5000; private static final double LRO_DEFAULT_POLL_DELAY_MULTIPLIER = 1.5; private static final long LRO_DEFAULT_MAX_POLL_DELAY_MILLIS = 45000; private static final long LRO_DEFAULT_TOTAL_POLL_TIMEOUT_MILLS = 86400000; // 24 hours. diff --git a/src/test/java/com/google/api/generator/gapic/composer/RetrySettingsComposerTest.java b/src/test/java/com/google/api/generator/gapic/composer/RetrySettingsComposerTest.java index 07fd8bbdc2..476592d4ac 100644 --- a/src/test/java/com/google/api/generator/gapic/composer/RetrySettingsComposerTest.java +++ b/src/test/java/com/google/api/generator/gapic/composer/RetrySettingsComposerTest.java @@ -356,7 +356,7 @@ public void lroBuilderExpr() { + "WaitResponse.class))" + ".setMetadataTransformer(ProtoOperationTransformers.MetadataTransformer.create(" + "WaitMetadata.class)).setPollingAlgorithm(OperationTimedPollAlgorithm.create(" - + "RetrySettings.newBuilder().setInitialRetryDelay(Duration.ofMillis(20000L))" + + "RetrySettings.newBuilder().setInitialRetryDelay(Duration.ofMillis(5000L))" + ".setRetryDelayMultiplier(1.5).setMaxRetryDelay(Duration.ofMillis(45000L))" + ".setInitialRpcTimeout(Duration.ZERO).setRpcTimeoutMultiplier(1.0)" + ".setMaxRpcTimeout(Duration.ZERO).setTotalTimeout(Duration.ofMillis(86400000L))" diff --git a/src/test/java/com/google/api/generator/gapic/composer/goldens/EchoStubSettings.golden b/src/test/java/com/google/api/generator/gapic/composer/goldens/EchoStubSettings.golden index 9bb853eee2..1cf711874b 100644 --- a/src/test/java/com/google/api/generator/gapic/composer/goldens/EchoStubSettings.golden +++ b/src/test/java/com/google/api/generator/gapic/composer/goldens/EchoStubSettings.golden @@ -415,7 +415,7 @@ public class EchoStubSettings extends StubSettings { .setPollingAlgorithm( OperationTimedPollAlgorithm.create( RetrySettings.newBuilder() - .setInitialRetryDelay(Duration.ofMillis(20000L)) + .setInitialRetryDelay(Duration.ofMillis(5000L)) .setRetryDelayMultiplier(1.5) .setMaxRetryDelay(Duration.ofMillis(45000L)) .setInitialRpcTimeout(Duration.ZERO) diff --git a/test/integration/goldens/redis/CloudRedisStubSettings.java b/test/integration/goldens/redis/CloudRedisStubSettings.java index 0ab9d31bae..5c4955ef25 100644 --- a/test/integration/goldens/redis/CloudRedisStubSettings.java +++ b/test/integration/goldens/redis/CloudRedisStubSettings.java @@ -552,7 +552,7 @@ private static Builder initDefaults(Builder builder) { .setPollingAlgorithm( OperationTimedPollAlgorithm.create( RetrySettings.newBuilder() - .setInitialRetryDelay(Duration.ofMillis(20000L)) + .setInitialRetryDelay(Duration.ofMillis(5000L)) .setRetryDelayMultiplier(1.5) .setMaxRetryDelay(Duration.ofMillis(45000L)) .setInitialRpcTimeout(Duration.ZERO) @@ -576,7 +576,7 @@ private static Builder initDefaults(Builder builder) { .setPollingAlgorithm( OperationTimedPollAlgorithm.create( RetrySettings.newBuilder() - .setInitialRetryDelay(Duration.ofMillis(20000L)) + .setInitialRetryDelay(Duration.ofMillis(5000L)) .setRetryDelayMultiplier(1.5) .setMaxRetryDelay(Duration.ofMillis(45000L)) .setInitialRpcTimeout(Duration.ZERO) @@ -600,7 +600,7 @@ private static Builder initDefaults(Builder builder) { .setPollingAlgorithm( OperationTimedPollAlgorithm.create( RetrySettings.newBuilder() - .setInitialRetryDelay(Duration.ofMillis(20000L)) + .setInitialRetryDelay(Duration.ofMillis(5000L)) .setRetryDelayMultiplier(1.5) .setMaxRetryDelay(Duration.ofMillis(45000L)) .setInitialRpcTimeout(Duration.ZERO) @@ -624,7 +624,7 @@ private static Builder initDefaults(Builder builder) { .setPollingAlgorithm( OperationTimedPollAlgorithm.create( RetrySettings.newBuilder() - .setInitialRetryDelay(Duration.ofMillis(20000L)) + .setInitialRetryDelay(Duration.ofMillis(5000L)) .setRetryDelayMultiplier(1.5) .setMaxRetryDelay(Duration.ofMillis(45000L)) .setInitialRpcTimeout(Duration.ZERO) @@ -648,7 +648,7 @@ private static Builder initDefaults(Builder builder) { .setPollingAlgorithm( OperationTimedPollAlgorithm.create( RetrySettings.newBuilder() - .setInitialRetryDelay(Duration.ofMillis(20000L)) + .setInitialRetryDelay(Duration.ofMillis(5000L)) .setRetryDelayMultiplier(1.5) .setMaxRetryDelay(Duration.ofMillis(45000L)) .setInitialRpcTimeout(Duration.ZERO) @@ -672,7 +672,7 @@ private static Builder initDefaults(Builder builder) { .setPollingAlgorithm( OperationTimedPollAlgorithm.create( RetrySettings.newBuilder() - .setInitialRetryDelay(Duration.ofMillis(20000L)) + .setInitialRetryDelay(Duration.ofMillis(5000L)) .setRetryDelayMultiplier(1.5) .setMaxRetryDelay(Duration.ofMillis(45000L)) .setInitialRpcTimeout(Duration.ZERO) @@ -696,7 +696,7 @@ private static Builder initDefaults(Builder builder) { .setPollingAlgorithm( OperationTimedPollAlgorithm.create( RetrySettings.newBuilder() - .setInitialRetryDelay(Duration.ofMillis(20000L)) + .setInitialRetryDelay(Duration.ofMillis(5000L)) .setRetryDelayMultiplier(1.5) .setMaxRetryDelay(Duration.ofMillis(45000L)) .setInitialRpcTimeout(Duration.ZERO) From b4a1ba9c42bcc7232ae9afa09170cd4505508c3f Mon Sep 17 00:00:00 2001 From: Mira Leung Date: Tue, 17 Nov 2020 14:43:50 -0800 Subject: [PATCH 2/4] fix: correct some common typos --- .../ClientLibraryPackageInfoComposer.java | 2 +- .../ResourceNameHelperClassComposer.java | 22 +++++++++---------- .../ServiceClientCommentComposer.java | 2 +- .../ServiceClientTestClassComposer.java | 2 +- .../ServiceStubSettingsClassComposer.java | 6 ++--- .../generator/gapic/protoparser/Parser.java | 2 +- .../ast/AssignmentOperationExprTest.java | 4 ++-- .../engine/ast/JavaDocCommentTest.java | 2 +- .../composer/DefaultValueComposerTest.java | 2 +- 9 files changed, 22 insertions(+), 22 deletions(-) diff --git a/src/main/java/com/google/api/generator/gapic/composer/ClientLibraryPackageInfoComposer.java b/src/main/java/com/google/api/generator/gapic/composer/ClientLibraryPackageInfoComposer.java index c4332baeb8..32f0c5b0f2 100644 --- a/src/main/java/com/google/api/generator/gapic/composer/ClientLibraryPackageInfoComposer.java +++ b/src/main/java/com/google/api/generator/gapic/composer/ClientLibraryPackageInfoComposer.java @@ -76,7 +76,7 @@ private static CommentStatement createPackageInfoJavadoc(GapicContext context) { javaDocCommentBuilder.addParagraph( String.format("%s %s %s", DIVIDER, javaClientName, DIVIDER)); - // TODO(miraleung): Replace this with a comment converter when we upport CommonMark. + // TODO(miraleung): Replace this with a comment converter when we support CommonMark. if (service.hasDescription()) { String[] descriptionParagraphs = service.description().split("\\n\\n"); for (int i = 0; i < descriptionParagraphs.length; i++) { diff --git a/src/main/java/com/google/api/generator/gapic/composer/ResourceNameHelperClassComposer.java b/src/main/java/com/google/api/generator/gapic/composer/ResourceNameHelperClassComposer.java index 00c8f03c34..543b297a66 100644 --- a/src/main/java/com/google/api/generator/gapic/composer/ResourceNameHelperClassComposer.java +++ b/src/main/java/com/google/api/generator/gapic/composer/ResourceNameHelperClassComposer.java @@ -1199,7 +1199,7 @@ private static MethodDefinition createEqualsMethod( Set tokenSet = getTokenSet(tokenHierarchies); Iterator itToken = tokenSet.iterator(); Expr curTokenExpr = - createObjectsEqualsForTokenMethodEpxr( + createObjectsEqualsForTokenMethodExpr( thisValueExpr, thatVariableExpr, Variable.builder() @@ -1208,7 +1208,7 @@ private static MethodDefinition createEqualsMethod( .build()); while (itToken.hasNext()) { Expr nextTokenExpr = - createObjectsEqualsForTokenMethodEpxr( + createObjectsEqualsForTokenMethodExpr( thisValueExpr, thatVariableExpr, Variable.builder() @@ -1252,7 +1252,7 @@ private static MethodDefinition createEqualsMethod( .build(); } - private static MethodInvocationExpr createObjectsEqualsForTokenMethodEpxr( + private static MethodInvocationExpr createObjectsEqualsForTokenMethodExpr( Expr thisExpr, Expr thatExpr, Variable tokenVar) { VariableExpr varThisExpr = VariableExpr.builder().setVariable(tokenVar).setExprReferenceExpr(thisExpr).build(); @@ -1267,7 +1267,7 @@ private static MethodInvocationExpr createObjectsEqualsForTokenMethodEpxr( } private static MethodDefinition createHashCodeMethod(List> tokenHierarchies) { - List asgmtBody = new ArrayList<>(); + List assignmentBody = new ArrayList<>(); // code: int h = 1; Variable hVar = Variable.builder().setType(TypeNode.INT).setName("h").build(); VariableExpr hVarExpr = VariableExpr.builder().setVariable(hVar).build(); @@ -1278,20 +1278,20 @@ private static MethodDefinition createHashCodeMethod(List> tokenHie .setVariableExpr(hVarExpr.toBuilder().setIsDecl(true).build()) .setValueExpr(hValueExpr) .build(); - asgmtBody.add(ExprStatement.withExpr(hAssignmentExpr)); + assignmentBody.add(ExprStatement.withExpr(hAssignmentExpr)); // code: h *= 1000003; // code: h ^= Objects.hashCode(...); ValueExpr numValueExpr = ValueExpr.withValue( PrimitiveValue.builder().setType(TypeNode.INT).setValue("1000003").build()); - AssignmentOperationExpr multiplyAsgmtOpExpr = + AssignmentOperationExpr multiplyAssignmentOpExpr = AssignmentOperationExpr.multiplyAssignmentWithExprs(hVarExpr, numValueExpr); // If it has variants, add the multiply and xor assignment operation exprs for fixedValue. boolean hasVariants = tokenHierarchies.size() > 1; if (hasVariants) { VariableExpr fixedValueVarExpr = FIXED_CLASS_VARS.get("fixedValue"); - asgmtBody.add(ExprStatement.withExpr(multiplyAsgmtOpExpr)); - asgmtBody.add( + assignmentBody.add(ExprStatement.withExpr(multiplyAssignmentOpExpr)); + assignmentBody.add( ExprStatement.withExpr( AssignmentOperationExpr.xorAssignmentWithExprs( hVarExpr, createObjectsHashCodeForVarMethod(fixedValueVarExpr)))); @@ -1307,8 +1307,8 @@ private static MethodDefinition createHashCodeMethod(List> tokenHie .setName(JavaStyle.toLowerCamelCase(token)) .setType(TypeNode.STRING) .build()); - asgmtBody.add(ExprStatement.withExpr(multiplyAsgmtOpExpr)); - asgmtBody.add( + assignmentBody.add(ExprStatement.withExpr(multiplyAssignmentOpExpr)); + assignmentBody.add( ExprStatement.withExpr( AssignmentOperationExpr.xorAssignmentWithExprs( hVarExpr, createObjectsHashCodeForVarMethod(tokenVarExpr)))); @@ -1319,7 +1319,7 @@ private static MethodDefinition createHashCodeMethod(List> tokenHie .setScope(ScopeNode.PUBLIC) .setReturnType(TypeNode.INT) .setName("hashCode") - .setBody(asgmtBody) + .setBody(assignmentBody) .setReturnExpr(hVarExpr) .build(); } diff --git a/src/main/java/com/google/api/generator/gapic/composer/ServiceClientCommentComposer.java b/src/main/java/com/google/api/generator/gapic/composer/ServiceClientCommentComposer.java index 1a0f8f2b0f..dfe76dffa2 100644 --- a/src/main/java/com/google/api/generator/gapic/composer/ServiceClientCommentComposer.java +++ b/src/main/java/com/google/api/generator/gapic/composer/ServiceClientCommentComposer.java @@ -217,7 +217,7 @@ private static CommentStatement toSimpleComment(String comment) { return CommentStatement.withComment(JavaDocComment.withComment(comment)); } - // TODO(miraleung): Replace this with a comment converter when we upport CommonMark. + // TODO(miraleung): Replace this with a comment converter when we support CommonMark. private static JavaDocComment.Builder processProtobufComment( String rawComment, JavaDocComment.Builder originalCommentBuilder, String firstPattern) { JavaDocComment.Builder commentBuilder = originalCommentBuilder; diff --git a/src/main/java/com/google/api/generator/gapic/composer/ServiceClientTestClassComposer.java b/src/main/java/com/google/api/generator/gapic/composer/ServiceClientTestClassComposer.java index 03033af23c..9e9119032c 100644 --- a/src/main/java/com/google/api/generator/gapic/composer/ServiceClientTestClassComposer.java +++ b/src/main/java/com/google/api/generator/gapic/composer/ServiceClientTestClassComposer.java @@ -657,7 +657,7 @@ private static MethodDefinition createRpcTestMethod( .build()); if (method.isPaged()) { - // Assign the resources variaqble. + // Assign the resources variable. VariableExpr resourcesVarExpr = VariableExpr.withVariable( Variable.builder() diff --git a/src/main/java/com/google/api/generator/gapic/composer/ServiceStubSettingsClassComposer.java b/src/main/java/com/google/api/generator/gapic/composer/ServiceStubSettingsClassComposer.java index c31bec5317..0c471392b5 100644 --- a/src/main/java/com/google/api/generator/gapic/composer/ServiceStubSettingsClassComposer.java +++ b/src/main/java/com/google/api/generator/gapic/composer/ServiceStubSettingsClassComposer.java @@ -433,7 +433,7 @@ private static Expr createPagedListDescriptorAssignExpr( VariableExpr pageSizeVarExpr = VariableExpr.withVariable( Variable.builder().setType(TypeNode.INT).setName("pageSize").build()); - // Re-declare for clarity and easier readeability. + // Re-declare for clarity and easier readability. returnType = method.inputType(); returnExpr = MethodInvocationExpr.builder() @@ -1431,7 +1431,7 @@ private static List createNestedClassConstructorMethods( .setBody(ctorBodyStatements) .build()); - // Third constructor that takes a ServivceStubSettings. + // Third constructor that takes a ServiceStubSettings. TypeNode outerSettingsType = types.get(getThisClassName(service.name())); VariableExpr settingsVarExpr = VariableExpr.withVariable( @@ -1882,7 +1882,7 @@ private static TypeNode getCallSettingsType( : typeMakerFn.apply( isSettingsBuilder ? UnaryCallSettings.Builder.class : UnaryCallSettings.class); - // Streaming takes precendence over paging, as per the monolith's existing behavior. + // Streaming takes precedence over paging, as per the monolith's existing behavior. switch (method.stream()) { case SERVER: callSettingsType = diff --git a/src/main/java/com/google/api/generator/gapic/protoparser/Parser.java b/src/main/java/com/google/api/generator/gapic/protoparser/Parser.java index b46c94cc9d..d3ed868bff 100644 --- a/src/main/java/com/google/api/generator/gapic/protoparser/Parser.java +++ b/src/main/java/com/google/api/generator/gapic/protoparser/Parser.java @@ -372,7 +372,7 @@ static List parseMethods( .setIsPaged(parseIsPaged(protoMethod, messageTypes)) .build()); - // Any input type that has a resource reference will need a resource name helper calss. + // Any input type that has a resource reference will need a resource name helper class. Message inputMessage = messageTypes.get(inputType.reference().name()); for (Field field : inputMessage.fields()) { if (field.hasResourceReference()) { diff --git a/src/test/java/com/google/api/generator/engine/ast/AssignmentOperationExprTest.java b/src/test/java/com/google/api/generator/engine/ast/AssignmentOperationExprTest.java index a3ef39c4c2..df767ed2f3 100644 --- a/src/test/java/com/google/api/generator/engine/ast/AssignmentOperationExprTest.java +++ b/src/test/java/com/google/api/generator/engine/ast/AssignmentOperationExprTest.java @@ -132,7 +132,7 @@ public void invalidMultiplyAndAssignmentOperationExpr_numericWithBooleanType() { @Test public void invalidMultiplyAndAssignmentOperationExpr_numericWithReferenceType() { - // Swap test case in "invalidMultiplyAndAssignmentOperationExpr_feferencedWithNumericType". + // Swap test case in "invalidMultiplyAndAssignmentOperationExpr_referencedWithNumericType". VariableExpr lhsExpr = createVariableExpr(TypeNode.INT, "x"); VariableExpr rhsExpr = createVariableExpr(TypeNode.STRING, "y"); assertThrows( @@ -470,7 +470,7 @@ public void invalidMultiplyAndAssignmentOperationExpr_doubleBoxedWithNullType() } @Test - public void invalidMultiplyAndAssignmentOperationExpr_doubleBoxedWithOjectType() { + public void invalidMultiplyAndAssignmentOperationExpr_doubleBoxedWithObjectType() { // No need swap test. VariableExpr lhsExpr = createVariableExpr(TypeNode.DOUBLE_OBJECT, "x"); NewObjectExpr rhsExpr = NewObjectExpr.withType(TypeNode.OBJECT); diff --git a/src/test/java/com/google/api/generator/engine/ast/JavaDocCommentTest.java b/src/test/java/com/google/api/generator/engine/ast/JavaDocCommentTest.java index 967b5ad575..3a27c10da9 100644 --- a/src/test/java/com/google/api/generator/engine/ast/JavaDocCommentTest.java +++ b/src/test/java/com/google/api/generator/engine/ast/JavaDocCommentTest.java @@ -172,7 +172,7 @@ public void createJavaDocComment_throwsAndDeprecated() { } @Test - public void createavaDocComment_allComponents() { + public void createJavaDocComment_allComponents() { // No matter what order `setThrows`, `setDeprecated` are called, // They will be printed at the end. And `@param` should be grouped, // they should always be printed right before `@throws` and `@deprecated`. diff --git a/src/test/java/com/google/api/generator/gapic/composer/DefaultValueComposerTest.java b/src/test/java/com/google/api/generator/gapic/composer/DefaultValueComposerTest.java index 0c397692b1..c0f67e9946 100644 --- a/src/test/java/com/google/api/generator/gapic/composer/DefaultValueComposerTest.java +++ b/src/test/java/com/google/api/generator/gapic/composer/DefaultValueComposerTest.java @@ -60,7 +60,7 @@ public void defaultValue_mapField() { writerVisitor.clear(); - // isMap() and isRepeated() will be set by protoc simulataneoulsy, but we check this edge case + // isMap() and isRepeated() will be set by protoc simultaneously, but we check this edge case. // for completeness. field = Field.builder().setName("foobar").setType(TypeNode.STRING).setIsMap(true).build(); expr = DefaultValueComposer.createDefaultValue(field); From 8c9d12dd3bfc323fa381b1a0da7730d6ed4f2633 Mon Sep 17 00:00:00 2001 From: Mira Leung Date: Tue, 17 Nov 2020 14:49:16 -0800 Subject: [PATCH 3/4] fix: switch assertEquals (actual,expected) args order in JavaDocCommentTest --- .../generator/engine/ast/JavaDocCommentTest.java | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/test/java/com/google/api/generator/engine/ast/JavaDocCommentTest.java b/src/test/java/com/google/api/generator/engine/ast/JavaDocCommentTest.java index 3a27c10da9..90a019c795 100644 --- a/src/test/java/com/google/api/generator/engine/ast/JavaDocCommentTest.java +++ b/src/test/java/com/google/api/generator/engine/ast/JavaDocCommentTest.java @@ -25,7 +25,7 @@ public class JavaDocCommentTest { public void createJavaDocComment_basic() { String content = "this is a test comment"; JavaDocComment javaDocComment = JavaDocComment.builder().addComment(content).build(); - assertEquals(javaDocComment.comment(), content); + assertEquals(content, javaDocComment.comment()); } @Test @@ -51,7 +51,7 @@ public void createJavaDocComment_specialCharacter() { + " libraryClient.createShelfCallable().futureCall(request);\n" + "}\n" + "@throws Exception This is an exception."; - assertEquals(javaDocComment.comment(), expected); + assertEquals(expected, javaDocComment.comment()); } @Test @@ -65,7 +65,7 @@ public void createJavaDocComment_sampleCode() { + "
{@code\n"
             + "resource = project/{project}/shelfId/{shelfId}\n"
             + "}
"; - assertEquals(javaDocComment.comment(), expected); + assertEquals(expected, javaDocComment.comment()); } @Test @@ -122,7 +122,7 @@ public void createJavaDocComment_multipleComments() { + "
  • A request object method.\n" + "
  • A callable method.\n" + ""; - assertEquals(javaDocComment.comment(), expected); + assertEquals(expected, javaDocComment.comment()); } @Test @@ -143,7 +143,7 @@ public void createJavaDocComment_multipleParams() { "This is a block comment.\n" + "@param shelfName The name of the shelf where books are published to.\n" + "@param shelfId The shelfId of the shelf where books are published to."; - assertEquals(javaDocComment.comment(), expected); + assertEquals(expected, javaDocComment.comment()); } @Test @@ -168,7 +168,7 @@ public void createJavaDocComment_throwsAndDeprecated() { String expected = "@throws java.lang.RuntimeException if the remote call fails.\n" + "@deprecated Use the {@link ShelfBookName} class instead."; - assertEquals(javaDocComment.comment(), expected); + assertEquals(expected, javaDocComment.comment()); } @Test @@ -219,7 +219,7 @@ public void createJavaDocComment_allComponents() { + "@param shelf The shelf to create.\n" + "@throws com.google.api.gax.rpc.ApiException if the remote call fails.\n" + "@deprecated Use the {@link ArchivedBookName} class instead."; - assertEquals(javaDocComment.comment(), expected); + assertEquals(expected, javaDocComment.comment()); } private static String createLines(int numLines) { From 7fdee882fa30674a0f0e737611c2fa6901fcb7a8 Mon Sep 17 00:00:00 2001 From: Mira Leung Date: Tue, 17 Nov 2020 15:30:01 -0800 Subject: [PATCH 4/4] fix: merge master --- test/integration/goldens/asset/AssetServiceStubSettings.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/integration/goldens/asset/AssetServiceStubSettings.java b/test/integration/goldens/asset/AssetServiceStubSettings.java index c535fa3857..bf1c0e2984 100644 --- a/test/integration/goldens/asset/AssetServiceStubSettings.java +++ b/test/integration/goldens/asset/AssetServiceStubSettings.java @@ -601,7 +601,7 @@ private static Builder initDefaults(Builder builder) { .setPollingAlgorithm( OperationTimedPollAlgorithm.create( RetrySettings.newBuilder() - .setInitialRetryDelay(Duration.ofMillis(20000L)) + .setInitialRetryDelay(Duration.ofMillis(5000L)) .setRetryDelayMultiplier(1.5) .setMaxRetryDelay(Duration.ofMillis(45000L)) .setInitialRpcTimeout(Duration.ZERO)