remove the default match arm in create_function_physical_name

Author: capkurmagati

datafusion/src/logical_plan/expr.rs

@@ -1988,13 +1988,12 @@ fn create_name(e: &Expr, input_schema: &DFSchema) -> Result<String> {
             }
         }
         Expr::Sort {
-            expr,
+            expr: _,
             asc: _,
             nulls_first: _,
-        } => Err(DataFusionError::Internal(format!(
-            "Create name does not support logical expression {:?}",
-            expr
-        ))),
+        } => Err(DataFusionError::Internal(
+            "Create name does not support sort expression".to_string(),
+        )),
         Expr::Wildcard => Err(DataFusionError::Internal(
             "Create name does not support wildcard".to_string(),
         )),

datafusion/src/physical_plan/planner.rs

@@ -197,10 +197,16 @@ fn create_physical_name(e: &Expr, is_first_expr: bool) -> Result<String> {
                 Ok(format!("{} BETWEEN {} AND {}", expr, low, high))
             }
         }
-        other => Err(DataFusionError::NotImplemented(format!(
-            "Cannot derive physical field name for logical expression {:?}",
-            other
-        ))),
+        Expr::Sort {
+            expr: _,
+            asc: _,
+            nulls_first: _,
+        } => Err(DataFusionError::Internal(
+            "Create physical name does not support sort expression".to_string(),
+        )),
+        Expr::Wildcard => Err(DataFusionError::Internal(
+            "Create physical name does not support wildcard".to_string(),
+        )),
     }
 }