You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2026/04/20 09:49:24 UTC
(doris) branch master updated: [opt](nerieds) PhysicalOlapScan and PhysicalFileScan should print its id in plan (#62509)
This is an automated email from the ASF dual-hosted git repository.
morrySnow pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 7c89384b796 [opt](nerieds) PhysicalOlapScan and PhysicalFileScan should print its id in plan (#62509)
7c89384b796 is described below
commit 7c89384b796188d34872cf650b9f6bf83d6fe810
Author: minghong <zh...@selectdb.com>
AuthorDate: Mon Apr 20 17:49:08 2026 +0800
[opt](nerieds) PhysicalOlapScan and PhysicalFileScan should print its id in plan (#62509)
---
.../apache/doris/nereids/trees/plans/physical/PhysicalFileScan.java | 5 +++--
.../apache/doris/nereids/trees/plans/physical/PhysicalHudiScan.java | 2 +-
.../apache/doris/nereids/trees/plans/physical/PhysicalOlapScan.java | 4 ++--
regression-test/suites/query_p0/stats/partitionRowCount.groovy | 4 ++--
4 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalFileScan.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalFileScan.java
index 520f03f6748..ca4e76bc2a8 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalFileScan.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalFileScan.java
@@ -137,8 +137,9 @@ public class PhysicalFileScan extends PhysicalCatalogRelation {
if (!runtimeFiltersV2.isEmpty()) {
rfV2 = runtimeFiltersV2.toString();
}
- return Utils.toSqlString("PhysicalFileScan[" + table.getName() + "]" + getGroupIdWithPrefix(),
- "stats", statistics,
+ return Utils.toSqlString("PhysicalFileScan[" + id.asInt() + "]" + getGroupIdWithPrefix(),
+ "table", table.getName(),
+ "stats", statistics,
"qualified", Utils.qualifiedName(qualifier, table.getName()),
"selected partitions num",
selectedPartitions.isPruned ? selectedPartitions.selectedPartitions.size() : "unknown",
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalHudiScan.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalHudiScan.java
index b1c99af04c4..13eee0bf1b9 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalHudiScan.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalHudiScan.java
@@ -120,7 +120,7 @@ public class PhysicalHudiScan extends PhysicalFileScan {
@Override
public String toString() {
- return Utils.toSqlString("PhysicalHudiScan",
+ return Utils.toSqlString("PhysicalHudiScan[" + id.asInt() + "]" + getGroupIdWithPrefix(),
"qualified", Utils.qualifiedName(qualifier, table.getName()),
"output", getOutput(),
"stats", statistics,
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalOlapScan.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalOlapScan.java
index 059a71ae745..f5e6c152a32 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalOlapScan.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalOlapScan.java
@@ -251,8 +251,8 @@ public class PhysicalOlapScan extends PhysicalCatalogRelation implements OlapSca
rfV2 = runtimeFiltersV2.toString();
}
- return Utils.toSqlString("PhysicalOlapScan[" + table.getName() + index + partitions + "]"
- + getGroupIdWithPrefix(),
+ return Utils.toSqlString("PhysicalOlapScan[" + id.asInt() + "]" + getGroupIdWithPrefix(),
+ "table", table.getName() + index + partitions,
"alias", tableAlias,
"stats", statistics,
"operativeSlots", operativeSlots,
diff --git a/regression-test/suites/query_p0/stats/partitionRowCount.groovy b/regression-test/suites/query_p0/stats/partitionRowCount.groovy
index 36433b8e415..c41cfd803f4 100644
--- a/regression-test/suites/query_p0/stats/partitionRowCount.groovy
+++ b/regression-test/suites/query_p0/stats/partitionRowCount.groovy
@@ -35,7 +35,7 @@ suite("partitionRowCount") {
sql """physical plan
select * from partitionRowCountTable where a < 250;
"""
- contains("PhysicalOlapScan[partitionRowCountTable partitions(2/3)]@0")
+ contains("PhysicalOlapScan[19]@0 ( table=partitionRowCountTable partitions(2/3)")
}
-}
\ No newline at end of file
+}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org