sleungtoast opened a new issue #4101: Fix Javadoc for ClientBuilder.keepAliveInterval URL: https://github.com/apache/pulsar/issues/4101 **Describe the bug** During the change in https://github.com/apache/pulsar/pull/1547, there was some copy and pasting that overlooked inconsistencies in variable naming and javadocs. ``` /** * Set keep alive interval in seconds for each client-broker-connection. <i>(default: 30)</i>. * * @param keepAliveIntervalSeconds * @param unit time unit for {@code statsInterval} * @return the client builder instance */ ClientBuilder keepAliveInterval(int keepAliveIntervalSeconds, TimeUnit unit); ``` `keepAliveIntervalSeconds` should now be `keepAliveInterval` in the javadoc and variable name until it's converted into seconds `@param unit time unit for {@code statsInterval}` should be `@param unit time unit for {@code keepAliveInterval}`
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
