fix(query/mem): release msg memory immediately instead of response#34988
fix(query/mem): release msg memory immediately instead of response#34988guanshengliang merged 4 commits intomainfrom
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a memory management improvement in source/libs/qcom/src/queryUtil.c by explicitly freeing pInfo->msgInfo.pData using taosMemoryFreeClear after its contents have been copied to a new buffer. I have no feedback to provide.
There was a problem hiding this comment.
Pull request overview
This PR addresses excessive memory retention during async RPC sends by freeing the original request message buffer immediately after it’s copied into the RPC-owned container, rather than holding it until the response/ahandle teardown.
Changes:
- Free
pInfo->msgInfo.pDataright aftermemcpy()intorpcMallocCont()buffer inasyncSendMsgToServerExt.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Description
Issue(s)
https://project.feishu.cn/taosdata_td/defect/detail/6880911034
Checklist
Please check the items in the checklist if applicable.