refactor(catalog): remove delete

[BohuTANG]

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

Remove the DeletePlan from catalog, and decouple the catalog from the plan:

    async fn delete(
        &self,
        ctx: Arc<dyn TableContext>,
        projection: &Projection,
        selection: &Option<String>,
    ) -> Result<()> {

Closes #issue

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Updated
databend	⬜️ Ignored (Inspect)		Nov 6, 2022 at 2:36PM (UTC)

[BohuTANG]

/// # TODO
///
/// From @xuanwo
///
/// Ideally, we need to use `Scalar` in DeletePlan.selection. But we met a
/// cycle deps here. So we have to change `selection` in String first, and
/// change into `Scalar` when our `Planner` has been moved out.
///
/// At this stage, DeletePlan's selection expr will be parsed twice:
///
/// - Parsed during `bind` to get column index and projection index.
/// - Parsed during `execution` to get the correct columns
///
/// It's an ugly but necessary price to pay. Without this, we would sink in
/// hell forever.

I think this TODO can TO-DO after this PR is merged, cc @Xuanwo

[BohuTANG]

@Xuanwo ,
I finally know the situation on the DeletePlan.selection, it's one step from the old-expression to the new-expression, now it's hard to do it, so lets the query&planner team do it :)

cc @leiysky @sundy-li