[GitHub] [pulsar] codelipenghui commented on issue #15859: PIP-172 : Introduce the `HEALTH_CHECK` command in the binary protocol

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

codelipenghui commented on issue #15859:
URL: https://github.com/apache/pulsar/issues/15859#issuecomment-1158781060

   >  required uint64 cost_time = 3;
   
   We can remove this one? The broker always returns the cached value, not a real publish latency
   
   > interface PulsarClient {
       // ....
       CompletableFuture<HealthCheckResult> healthCheck();
   }
   
   Could you please add some details about how this method will be used?
   It will check all the clusters or the connected cluster?
   
   > interface LookupService {
     //......
     CompletableFuture<HealthCheckResult> healthCheck();
   }
   
   Could you please add some context about why add this method to LookupService?
   
   > Caching healthCheck result 60s on boeker side.
   
   It should be configurable.


-- 
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