Closed iajoiner closed 1 month 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.
!
source scripts/run_ci_checks.sh
We need to add ColumnarValue so that we can remove table_length from ProofExpr::result_evaluate which helps with ProofPlan composition.
ColumnarValue
table_length
ProofExpr::result_evaluate
ProofPlan
base::database::ColumnarValue
Yes
Just rebasing. No changes since @JayWhite2357 's last approval.
:tada: This PR is included in version 0.35.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:
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
!
is used if and only if at least one breaking change has been introduced.source scripts/run_ci_checks.sh
.Rationale for this change
We need to add
ColumnarValue
so that we can removetable_length
fromProofExpr::result_evaluate
which helps withProofPlan
composition.What changes are included in this PR?
base::database::ColumnarValue
Yes