idea: add another `static_name()` method to `ExecutionPlan` trait about arrow-datafusion HOT 4 CLOSED

cc @matthewmturner and @comphead from #9793

from arrow-datafusion.

I have a similar pattern on implementing user defined logical plan:

https://github.com/GreptimeTeam/greptimedb/blob/4eadd9f8a88870fff72922d9c7515cf29bc790b3/src/promql/src/extension_plan/empty_metric.rs#L130-L133

where

impl UserDefinedLogicalNodeCore for EmptyMetric {
    fn name(&self) -> &str {
        Self::name()
    }
}

impl EmptyMetricExec {
    pub const fn name() -> &'static str {
        "EmptyMetric"
    }
}

from arrow-datafusion.

Thanks @waynexia could you please give an example how the static be called for different plan nodes to obtain different names?

from arrow-datafusion.

Thanks @waynexia could you please give an example how the static be called for different plan nodes to obtain different names?

This (old) test case can be an example: https://github.com/apache/datafusion/pull/10266/files#diff-9f1cf112a3f7c21baab307b9001331d287479e54e60b594712395052e0ad0690R935-R936

After #10266 the call to ExecutionPlan::name() is proxied to static_name() in default implementation.

from arrow-datafusion.