Skip to content

Commit

Permalink
[chore](scan) remove useless enable_scan_node_run_serial (apache#41559)
Browse files Browse the repository at this point in the history
enable_scan_node_run_serial is not working after
apache#39927
  • Loading branch information
zhiqiang-hhhh authored and zhiqiang-hhhh committed Nov 11, 2024
1 parent 91d95f6 commit cbf0266
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 19 deletions.
5 changes: 0 additions & 5 deletions be/src/runtime/runtime_state.h
Original file line number Diff line number Diff line change
Expand Up @@ -460,11 +460,6 @@ class RuntimeState {
return _query_options.__isset.enable_profile && _query_options.enable_profile;
}

bool enable_scan_node_run_serial() const {
return _query_options.__isset.enable_scan_node_run_serial &&
_query_options.enable_scan_node_run_serial;
}

bool enable_share_hash_table_for_broadcast_join() const {
return _query_options.__isset.enable_share_hash_table_for_broadcast_join &&
_query_options.enable_share_hash_table_for_broadcast_join;
Expand Down
12 changes: 0 additions & 12 deletions fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java
Original file line number Diff line number Diff line change
Expand Up @@ -484,8 +484,6 @@ public class SessionVariable implements Serializable, Writable {

public static final String ENABLE_CTE_MATERIALIZE = "enable_cte_materialize";

public static final String ENABLE_SCAN_RUN_SERIAL = "enable_scan_node_run_serial";

public static final String ENABLE_ANALYZE_COMPLEX_TYPE_COLUMN = "enable_analyze_complex_type_column";

public static final String EXTERNAL_TABLE_ANALYZE_PART_NUM = "external_table_analyze_part_num";
Expand Down Expand Up @@ -995,12 +993,6 @@ public enum IgnoreSplitType {
@VariableMgr.VarAttr(name = ENABLE_ODBC_TRANSCATION)
public boolean enableOdbcTransaction = false;

@VariableMgr.VarAttr(name = ENABLE_SCAN_RUN_SERIAL, description = {
"是否开启ScanNode串行读,以避免limit较小的情况下的读放大,可以提高查询的并发能力",
"Whether to enable ScanNode serial reading to avoid read amplification in cases of small limits"
+ "which can improve query concurrency. default is false."})
public boolean enableScanRunSerial = false;

@VariableMgr.VarAttr(name = ENABLE_SQL_CACHE)
public boolean enableSqlCache = false;

Expand Down Expand Up @@ -3104,10 +3096,6 @@ public void setShowHiddenColumns(boolean showHiddenColumns) {
this.showHiddenColumns = showHiddenColumns;
}

public boolean isEnableScanRunSerial() {
return enableScanRunSerial;
}

public boolean skipStorageEngineMerge() {
return skipStorageEngineMerge;
}
Expand Down
4 changes: 2 additions & 2 deletions gensrc/thrift/PaloInternalService.thrift
Original file line number Diff line number Diff line change
Expand Up @@ -226,8 +226,8 @@ struct TQueryOptions {
72: optional bool enable_orc_lazy_mat = true

73: optional i64 scan_queue_mem_limit

74: optional bool enable_scan_node_run_serial = false;
// deprecated
74: optional bool enable_scan_node_run_serial = false;

75: optional bool enable_insert_strict = false;

Expand Down

0 comments on commit cbf0266

Please sign in to comment.