felipecrv commented on PR #41320: URL: https://github.com/apache/arrow/pull/41320#issuecomment-2075058403
> Also cc @felipecrv , do you think TableReader would check the input is valid? I think the generate side should checks it, and the consumer would better dcheck that? Unless the inputs come from an external source (eg IPC), we shouldn't be validating them in `Release` builds — validation with `DCHECK` is OK. We should fix the producer of the invalid `Table` in this case. Is the Parquet reader producing an invalid `Table` instance and passing it to `TableReader`? -- 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...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
