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/11 20:44:16 UTC

[GitHub] [pulsar] nicoloboschi opened a new pull request, #15125: [fix][pulsar-io] Fix memory leak in ElasticSearch sink

nicoloboschi opened a new pull request, #15125:
URL: https://github.com/apache/pulsar/pull/15125

   ### Motivation
   After https://github.com/apache/pulsar/pull/14805, there's a memory leak while using the OpenSearch client.
   Every request is kept in a collection, mapping the request with an unique id. The collection is never clear and the objects never collected by the GC.
   
   ### Modifications
   
   * Refactor the internal mapping needed for access the pulsar record removing the collection and passing the object directly into the callback (using a wrapper object)
   
   Note: this only affects current pulsar master. 
   
   - [x] `no-need-doc` 


-- 
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] nicoloboschi commented on pull request #15125: [fix][pulsar-io] Fix memory leak in ElasticSearch sink

Posted by GitBox <gi...@apache.org>.
nicoloboschi commented on PR #15125:
URL: https://github.com/apache/pulsar/pull/15125#issuecomment-1098940372

   @eolivelli @codelipenghui @lhotari @dlg99 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] nicoloboschi merged pull request #15125: [fix][pulsar-io] Fix memory leak in ElasticSearch sink

Posted by GitBox <gi...@apache.org>.
nicoloboschi merged PR #15125:
URL: https://github.com/apache/pulsar/pull/15125


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