alamb commented on issue #16994: URL: https://github.com/apache/datafusion/issues/16994#issuecomment-3141279920
> What might be useful is to have more general test logic that takes in an ExecutionPlan and verifies (to the extent possible) that the post conditions required by the trait are met. For instance that the length of the various Vecs is correct. This can't check for correctness of the values (i.e. true or false in this case), but it can check for wellformedness of the result. @wiedld started adding something like this with the `ExecutionPlan::check_invariants`: https://docs.rs/datafusion/latest/datafusion/physical_plan/trait.ExecutionPlan.html#method.check_invariants Maybe we can use that mechanism? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
