Re: [PR] [fix](coalesce) fix 'heap-use-after-free' of function coalesce [doris]

2024-10-28 Thread via GitHub
hello-stephen merged PR #42666: URL: https://github.com/apache/doris/pull/42666 -- 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...@dori

Re: [PR] [fix](coalesce) fix 'heap-use-after-free' of function coalesce [doris]

2024-10-28 Thread via GitHub
github-actions[bot] commented on PR #42666: URL: https://github.com/apache/doris/pull/42666#issuecomment-2443306410 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] [fix](coalesce) fix 'heap-use-after-free' of function coalesce [doris]

2024-10-28 Thread via GitHub
github-actions[bot] commented on PR #42666: URL: https://github.com/apache/doris/pull/42666#issuecomment-2443306449 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] [fix](coalesce) fix 'heap-use-after-free' of function coalesce [doris]

2024-10-28 Thread via GitHub
github-actions[bot] commented on PR #42666: URL: https://github.com/apache/doris/pull/42666#issuecomment-2443032342 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [fix](coalesce) fix 'heap-use-after-free' of function coalesce [doris]

2024-10-28 Thread via GitHub
jacktengg commented on PR #42666: URL: https://github.com/apache/doris/pull/42666#issuecomment-2443028292 run buildall -- 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 un

Re: [PR] [fix](coalesce) fix 'heap-use-after-free' of function coalesce [doris]

2024-10-28 Thread via GitHub
doris-robot commented on PR #42666: URL: https://github.com/apache/doris/pull/42666#issuecomment-2443028213 Thank you for your contribution to Apache Doris. Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+y

[PR] [fix](coalesce) fix 'heap-use-after-free' of function coalesce [doris]

2024-10-28 Thread via GitHub
jacktengg opened a new pull request, #42666: URL: https://github.com/apache/doris/pull/42666 ## Proposed changes Should not modify state of vexpr when executing the expr, since it may be executed in parallel. Issue introduced by #42136 ``` ==75397==ERROR: AddressSan