lhotari commented on code in PR #23222: URL: https://github.com/apache/pulsar/pull/23222#discussion_r1736050474
########## pip/pip-375.md: ########## @@ -0,0 +1,32 @@ +# PIP-375: Expose the Admin client configs: readTimeout, requestTimeout, connectionTimeout + +# Background knowledge + +- `pulsar-admin sinks create` will upload files to brokers, but if the file is large and the network is slow, it may cost a long time. +- The default value of `requestTimeout` of Admin API is `60s`, after the fix https://github.com/apache/pulsar/pull/23128, it was changed to `300s`. Review Comment: Do you mean to say that the default *was* 60s before the change in PR 23128, which made the default value 300s? Why was this increase in the value a problem? Or was it a problem? -- 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]
