sxt-proof-of-sql icon indicating copy to clipboard operation
sxt-proof-of-sql copied to clipboard

feat!: allow `ProjectionExec` to take general inputs

Open iajoiner opened this issue 3 months ago • 1 comments

Please be sure to look over the pull request guidelines here: https://github.com/spaceandtimelabs/sxt-proof-of-sql/blob/main/CONTRIBUTING.md#submit-pr.

Please go through the following checklist

  • [x] The PR title and commit messages adhere to guidelines here: https://github.com/spaceandtimelabs/sxt-proof-of-sql/blob/main/CONTRIBUTING.md. In particular ! is used if and only if at least one breaking change has been introduced.
  • [x] I have run the ci check script with source scripts/run_ci_checks.sh.

Rationale for this change

We need to make it possible to compose ProjectionExec with ProofPlans.

What changes are included in this PR?

  • replace TableExpr in ProjectionExec with Box<DynProofPlan>

Are these changes tested?

Yes.

iajoiner avatar Nov 20 '24 14:11 iajoiner