zhijiangW edited a comment on issue #11453: [FLINK-13553][tests] Fix ByteBuf 
leak in KvStateServerHandlerTest
URL: https://github.com/apache/flink/pull/11453#issuecomment-601594528
 
 
   Thanks for the clarification. If I guess correctly, there are no limitations 
inside netty allocator to request memory unless direct OOM. That means even 
thought some requested `ByteBuf` leak, it would block any following allocation 
requests.
   
   Also the netty stack has some mechanism to check whether the requested 
`ByteBuf` is leaked. There is a similar case in 
https://issues.apache.org/jira/browse/FLINK-16582.
   
   I agree to launch a separate ticket for this leak fix. I have no other 
concerns now. LGTM! 
----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to