HappenLee commented on pull request #4135:
URL: https://github.com/apache/incubator-doris/pull/4135#issuecomment-662322631


   Hello, I have two questions to ask。
   1. shared ptr use for avoid memory leaks. but at the same time, it brings 
about several problems:
     (1)There are additional costs of ref count of ptr
     (2)  The interface becomes infectious
   So Is it good enough to do this replacement of shared_ptr ? what benefit us?
   
   2. ```unique_ptr``` runtime expenses is close to the raw pointer and it's 
simple enough to use。
   In most cases, should we replacing it with ```shared_prt```.
   
   
   
   
   
   
   
    


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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to