Skip to content

Commit dfd2251

Browse files
engleflyYour Name
authored andcommitted
[fix](nereids)relase physical plan in Profile.releaseMemory() (#57257)
### What problem does this PR solve? The profile holds a physical plan object. In profile.releaseMemory(), the physicalPlan needs to be set to null to release it, preventing the groupExpression in Memo from failing to be released in a timely manner. Issue Number: close #xxx Related PR: #xxx Problem Summary: ### Release note None ### Check List (For Author) - Test <!-- At least one of them must be included. --> - [ ] Regression test - [ ] Unit Test - [ ] Manual test (add detailed scripts or steps below) - [ ] No need to test or manual test. Explain why: - [ ] This is a refactor/code format and no logic has been changed. - [ ] Previous test can cover this change. - [ ] No code files have been changed. - [ ] Other reason <!-- Add your reason? --> - Behavior changed: - [ ] No. - [ ] Yes. <!-- Explain the behavior change --> - Does this need documentation? - [ ] No. - [ ] Yes. <!-- Add document PR link here. eg: apache/doris-website#1214 --> ### Check List (For Reviewer who merge this PR) - [ ] Confirm the release note - [ ] Confirm test cases - [ ] Confirm document - [ ] Add branch pick label <!-- Add branch pick label that this PR should merge into -->
1 parent 5ed06bb commit dfd2251

File tree

1 file changed

+3
-0
lines changed
  • fe/fe-core/src/main/java/org/apache/doris/common/profile

1 file changed

+3
-0
lines changed

fe/fe-core/src/main/java/org/apache/doris/common/profile/Profile.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -434,6 +434,9 @@ public void setSummaryProfile(SummaryProfile summaryProfile) {
434434
public void releaseMemory() {
435435
this.executionProfiles.clear();
436436
this.changedSessionVarCache = "";
437+
this.physicalPlan = null;
438+
this.rowsProducedMap = null;
439+
this.physicalRelations = null;
437440
}
438441

439442
public boolean shouldStoreToStorage() {

0 commit comments

Comments
 (0)