dnrusakov opened a new pull request #3230: memory leak in tmaster fixed
URL: https://github.com/apache/incubator-heron/pull/3230
 
 
   The memory leak in TMaster was reported by one of the Twitter internal 
clients.
   After running TMaster under Valgrind memory leaks detector, the leak was 
localized.
   This PR aims to fix it. 
   
   `GetStmgrsRegSummary()` now returns a smart pointer (`unique_ptr<>`). So, as 
soon as a caller no need the returned result anymore the memory is freed 
automatically.
   
   Tested this PR locally under Valgrind and don't see the leak anymore.

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


With regards,
Apache Git Services

Reply via email to