You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/09/23 10:27:47 UTC
[GitHub] [pulsar] coderzc opened a new pull request, #17821: [fix][doc] Fix maxNumberOfRejectedRequestPerConnection doc
coderzc opened a new pull request, #17821:
URL: https://github.com/apache/pulsar/pull/17821
### Motivation
> The maximum number of rejected requests of a broker in a certain time frame (30 seconds) after the current connection is closed and the client creates a new connection to connect to a different broker.
The time frame should be in the 60s instead of the 30s, see:
https://github.com/apache/pulsar/blob/7913fe5614fed853f91199f658fe2584f10e38b9/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java#L157
https://github.com/apache/pulsar/blob/7913fe5614fed853f91199f658fe2584f10e38b9/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java#L1134-L1145
### Modifications
30 seconds -> 60seconds
### Documentation
- [x] `doc`
(Your PR contains doc changes)
--
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
[GitHub] [pulsar] coderzc commented on pull request #17821: [fix][doc] Fix maxNumberOfRejectedRequestPerConnection doc
Posted by GitBox <gi...@apache.org>.
coderzc commented on PR #17821:
URL: https://github.com/apache/pulsar/pull/17821#issuecomment-1257654519
@Anonymitaet @momo-jun PTAL
--
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
[GitHub] [pulsar] momo-jun merged pull request #17821: [fix][doc] Fix maxNumberOfRejectedRequestPerConnection doc
Posted by GitBox <gi...@apache.org>.
momo-jun merged PR #17821:
URL: https://github.com/apache/pulsar/pull/17821
--
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
[GitHub] [pulsar] momo-jun commented on pull request #17821: [fix][doc] Fix maxNumberOfRejectedRequestPerConnection doc
Posted by GitBox <gi...@apache.org>.
momo-jun commented on PR #17821:
URL: https://github.com/apache/pulsar/pull/17821#issuecomment-1257686221
@coderzc thanks for maintaining the docs. Can you pls also include the update in 2.8.x docs and invite a peer to review the PR for technical validity?
--
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
[GitHub] [pulsar] coderzc commented on pull request #17821: [fix][doc] Fix maxNumberOfRejectedRequestPerConnection doc
Posted by GitBox <gi...@apache.org>.
coderzc commented on PR #17821:
URL: https://github.com/apache/pulsar/pull/17821#issuecomment-1257785296
> @coderzc thanks for maintaining the docs. Can you pls also include the update in 2.8.x docs and invite a peer to review the PR for technical validity?
I updated it 2.8.x docs.
--
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