wip: cleanup

Author: matthewgapp

datafusion/core/src/physical_planner.rs

@@ -1335,7 +1335,6 @@ impl DefaultPhysicalPlanner {
                         Ok(plan)
                     }
                 }
-                // LogicalPlan::SubqueryAlias(SubqueryAlias())
                 LogicalPlan::RecursiveQuery(RecursiveQuery { name, static_term, recursive_term, is_distinct }) => {
                     let name = format!("{}-{}", name, new_recursive_cte_physical_plan_branch_number());
 

datafusion/sql/tests/sql_integration.rs

@@ -1387,28 +1387,6 @@ fn select_interval_out_of_range() {
     );
 }
 
-#[test]
-fn select_array_no_common_type() {
-    let sql = "SELECT [1, true, null]";
-    let err = logical_plan(sql).expect_err("query should have failed");
-
-    // HashSet doesn't guarantee order
-    assert_contains!(
-        err.strip_backtrace(),
-        "This feature is not implemented: Arrays with different types are not supported: "
-    );
-}
-
-#[test]
-fn select_array_non_literal_type() {
-    let sql = "SELECT [now()]";
-    let err = logical_plan(sql).expect_err("query should have failed");
-    assert_eq!(
-        "This feature is not implemented: Arrays with elements other than literal are not supported: now()",
-        err.strip_backtrace()
-    );
-}
-
 #[test]
 fn select_simple_aggregate_with_groupby_and_column_is_in_aggregate_and_groupby() {
     quick_test(