morningman opened a new pull request, #23638:
URL: https://github.com/apache/doris/pull/23638

   ## Proposed changes
   
   When file meta cache is disabled (by setting 
`max_external_file_meta_cache_num=0` in be.conf),
   the parquet's meta info is owned by parquet reader and will be released when 
calling `reader->close()`.
   
   But the underlying file reader of this parquet reader will be released after 
`reader->close()`,
   this may causing `heap-use-after-free` bug because some part of meta info 
may be referenced by file reader.
   
   This PR fix it by making sure that meta info is released after file reader 
released.
   
   ## Further comments
   
   If this is a relatively large or complex change, kick off the discussion at 
[d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you 
chose the solution you did and what alternatives you considered, etc...
   
   


-- 
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-unsubscr...@doris.apache.org

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