walkinggo commented on code in PR #23989:
URL: https://github.com/apache/pulsar/pull/23989#discussion_r2421057005
##########
pulsar-client-api/src/main/java/org/apache/pulsar/client/api/ClientBuilder.java:
##########
@@ -718,6 +718,13 @@ ClientBuilder authentication(String authPluginClassName,
Map<String, String> aut
*/
ClientBuilder lookupProperties(Map<String, String> properties);
+ /**
+ * Set the max retry times for a request.
+ *
+ * @param maxRetryTimes
+ * @return the client builder instance
+ */
+ ClientBuilder maxRetryTimes(int maxRetryTimes);
Review Comment:
> The Pulsar Admin Builder implementation also uses the
ClientConfigurationData to hold configuration data (which isn't the greatest
solution since not all configuration applies to both ClientBuilder and
PulsarAdminBuilder).
I think this is also a part that needs to be revised, and I will try to
clarify their relationships.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
