spaceandtimelabs / sxt-proof-of-sql

Space and Time | Proof of SQL

Other

2.74k stars 96 forks source link

feat!: allow `ProjectionExec` to take general inputs #384

Open iajoiner opened 1 week ago

iajoiner commented 1 week ago

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>
Yes.

iajoiner commented 1 week ago

Replacing #377 since GitHub was stuck