[GitHub] [pulsar] codelipenghui commented on a change in pull request #12520: [pulsar-client] Add conf backoff values

[GitBox <gi...@apache.org>]

codelipenghui commented on a change in pull request #12520:
URL: https://github.com/apache/pulsar/pull/12520#discussion_r744066079



##########
File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PulsarClientImpl.java
##########
@@ -884,9 +884,9 @@ public void reloadLookUp() throws PulsarClientException {
             TopicName topicName = TopicName.get(topic);
             AtomicLong opTimeoutMs = new AtomicLong(conf.getLookupTimeoutMs());
             Backoff backoff = new BackoffBuilder()
-                    .setInitialTime(100, TimeUnit.MILLISECONDS)
+                    .setInitialTime(conf.getInitialBackoffIntervalNanos, TimeUnit.NANOSECONDS)

Review comment:
       Nice catch, looks like we apply the configuration for producer and consumer but missed the client part.
   
   Could you please help add a test for covering the changes to make sure we will not make a regression in the future?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org