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 2020/09/28 06:36:43 UTC

[GitHub] [pulsar] lhotari commented on issue #8138: [Pulsar Java Client] Reader API usage with a lot of shortly used Reader instances (created-used-closed) under heavy load causes the client's memory consumption to grow until there is an OOM

lhotari commented on issue #8138:
URL: https://github.com/apache/pulsar/issues/8138#issuecomment-699806142


   > Would you like to provide a Pr to fix this.
   
   I'll try to come up with a fix. I'm thinking of adding timeout handling also for the other type of requests (GetLastMessageIdRequests, GetTopicsRequests & GetSchemaRequests) that don't have timeout handling in ClientCnx class. Unifying the way timeouts are handled for requests is something I'm thinking of. Instead of adding new ways to handle timeouts, the existing requestTimeoutQueue can be extended to support other type of requests.
   https://github.com/apache/pulsar/blob/754b864cf5f8844881eb9d47f4eaba6b4fb6d5c2/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java#L1021-L1038
   
   


----------------------------------------------------------------
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:
users@infra.apache.org