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/04/27 11:26:28 UTC

[GitHub] [pulsar] BewareMyPower commented on pull request #15321: [improve][client] Should use CompletableFuture#get(long, TimeUnit)

BewareMyPower commented on PR #15321:
URL: https://github.com/apache/pulsar/pull/15321#issuecomment-1110887488

   > it is very convenient to upgrade jdk on the server.
   
   It's controversial. If yes, there won't be any concern to upgrade Pulsar's source code to JDK17 compatible.
   
   > There will be performance problems in the scenario of high concurrent calls.
   
   In which case will the user call synchronized method instead of the **asynchronous** method if you cared about the performance (latency, throughput)? The performance overhead for a synchronous method is really a little thing if being compared with the performance gap between `sendAsync` and `send` calls.


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