Nayana-Naik73 commented on PR #53521: URL: https://github.com/apache/arrow/pull/53521#issuecomment-6232492072
@Reranko05 fair enough, I've opened #54025 for it and retitled the PR to match. @dmitry-chirkov-dremio added ReadRelWithOutOfBoundsEmit and ProjectRelWithOutOfBoundsEmit to serde_test.cc. Each one feeds a negative and an out-of-range outputMapping index through DeserializePlans and expects Status::Invalid, so both the generic GetEmitInfo path and ProcessEmitProject are covered. I checked them against the unfixed code too, where both trip the Schema::field DCHECK in a debug build. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
