Skip to content

Commit

Permalink
[Refactor] Remove unused session variable 'profile_limit_fold' (StarR…
Browse files Browse the repository at this point in the history
…ocks#33997)

Signed-off-by: liuyehcf <[email protected]>
Signed-off-by: Moonm3n <[email protected]>
  • Loading branch information
liuyehcf authored and Moonm3n committed Oct 31, 2023
1 parent 757d8af commit 02af40b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 14 deletions.
12 changes: 0 additions & 12 deletions fe/fe-core/src/main/java/com/starrocks/qe/SessionVariable.java
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,6 @@ public class SessionVariable implements Serializable, Writable, Cloneable {

public static final String PROFILE_TIMEOUT = "profile_timeout";
public static final String RUNTIME_PROFILE_REPORT_INTERVAL = "runtime_profile_report_interval";
public static final String PROFILE_LIMIT_FOLD = "profile_limit_fold";
public static final String PIPELINE_PROFILE_LEVEL = "pipeline_profile_level";
public static final String ENABLE_ASYNC_PROFILE = "enable_async_profile";

Expand Down Expand Up @@ -839,9 +838,6 @@ public static MaterializedViewRewriteMode parse(String str) {
@VariableMgr.VarAttr(name = RUNTIME_PROFILE_REPORT_INTERVAL)
private int runtimeProfileReportInterval = 10;

@VariableMgr.VarAttr(name = PROFILE_LIMIT_FOLD, flag = VariableMgr.INVISIBLE)
private boolean profileLimitFold = true;

@VariableMgr.VarAttr(name = PIPELINE_PROFILE_LEVEL)
private int pipelineProfileLevel = 1;

Expand Down Expand Up @@ -2186,14 +2182,6 @@ public int getRuntimeProfileReportInterval() {
return runtimeProfileReportInterval;
}

public void setProfileLimitFold(boolean profileLimitFold) {
this.profileLimitFold = profileLimitFold;
}

public boolean isProfileLimitFold() {
return profileLimitFold;
}

public void setPipelineProfileLevel(int pipelineProfileLevel) {
this.pipelineProfileLevel = pipelineProfileLevel;
}
Expand Down
2 changes: 0 additions & 2 deletions fe/fe-core/src/main/java/com/starrocks/qe/StmtExecutor.java
Original file line number Diff line number Diff line change
Expand Up @@ -940,7 +940,6 @@ private void handleQueryStmt(ExecPlan execPlan) throws Exception {
context.getSessionVariable().setEnableProfile(true);
context.getSessionVariable().setEnableAsyncProfile(false);
context.getSessionVariable().setPipelineProfileLevel(1);
context.getSessionVariable().setProfileLimitFold(false);
} else if (isSchedulerExplain) {
// Do nothing.
} else if (parsedStmt.isExplain()) {
Expand Down Expand Up @@ -1722,7 +1721,6 @@ public void handleDMLStmt(ExecPlan execPlan, DmlStmt stmt) throws Exception {
context.getSessionVariable().setEnableProfile(true);
context.getSessionVariable().setEnableAsyncProfile(false);
context.getSessionVariable().setPipelineProfileLevel(1);
context.getSessionVariable().setProfileLimitFold(false);
} else if (isSchedulerExplain) {
// Do nothing.
} else if (stmt.isExplain()) {
Expand Down

0 comments on commit 02af40b

Please sign in to comment.