alamb commented on PR #13187: URL: https://github.com/apache/datafusion/pull/13187#issuecomment-2453403770
> Thanks for the fix. LGTM! > > I wonder we could and some debug_assert or something to catch bugs of this sort in some general way. This is an interesting idea @eejbyfeldt . Would the debug assert be implemented on statistics? I suppose we could potentially implement a function that verifies that the output of `ExecutionPlan::statistics` has the same number of columns as `ExecutionPlan::schema()` and that the types matched 🤔 -- 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
