lucasbru commented on PR #12809:
URL: https://github.com/apache/kafka/pull/12809#issuecomment-1331835870

   I think the new version of RocksDB, or the way it was used in this PR, is 
leaking memory. To the point that with this change, a stream thread will only 
survive a few hours without running out of memory.
   
   I don't have a native heap profile yet, but this is the heap consumption in 
the first 8 hours of running trunk:
   
   ![Screenshot 2022-11-30 at 09 59 
15](https://user-images.githubusercontent.com/1628637/204752317-ac33cdf4-9ab8-4b5c-9964-29f7e0246f55.png)
   
   This is the heap consumption of trunk after reverting this change ( 
lucaasbru/kafka branch revert ):
   
   ![Screenshot 2022-11-30 at 10 00 
23](https://user-images.githubusercontent.com/1628637/204752541-b0bbd9e1-d263-4ef3-a369-f3de9ee4bac2.png)
   
   Can we revert this change in trunk until we have resolved this issue? 
@cadonna @clolov @ableegoldman 


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to