Skip to content

[bugfix](expr) fix memory leak in expr - #9820

Merged
morningman merged 1 commit into
apache:dev-1.0.1from
yixiutt:vexpr_memory_leak
May 27, 2022
Merged

[bugfix](expr) fix memory leak in expr#9820
morningman merged 1 commit into
apache:dev-1.0.1from
yixiutt:vexpr_memory_leak

Conversation

@yixiutt

Copy link
Copy Markdown
Contributor

too big to merge to 1.0.1

Proposed changes

Issue Number: close #xxx

Problem Summary:

Describe the overview of changes.

Checklist(Required)

  1. Does it affect the original behavior: (Yes/No/I Don't know)
  2. Has unit tests been added: (Yes/No/No Need)
  3. Has document been added or modified: (Yes/No/No Need)
  4. Does it need to update dependencies: (Yes/No)
  5. Are there any changes that cannot be rolled back: (Yes/No)

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@morningmanmorningman added the dev/1.0.1-deprecated should be merged into dev-1.0.1 branch label May 27, 2022
@yixiuttyixiutt changed the title [bugfix](expr) fix memory leak in expr, already fix in pr #9280 which is[bugfix](expr) fix memory leak in exprMay 27, 2022
already fix in pr apache#9280 but too big to merge to 1.0.1
@yixiutt
yixiuttforce-pushed the vexpr_memory_leak branch from b202b79 to 47d30b4CompareMay 27, 2022 15:05
@morningman
morningman merged commit 96ad450 into apache:dev-1.0.1May 27, 2022
@morningmanmorningman added dev/merged-1.0.1-deprecated PR has been merged into dev-1.0.1 and removed dev/1.0.1-deprecated should be merged into dev-1.0.1 branch labels May 27, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev/merged-1.0.1-deprecatedPR has been merged into dev-1.0.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@yixiutt@morningman