metsw24-max commented on PR #50070: URL: https://github.com/apache/arrow/pull/50070#issuecomment-4715380908
Added regression tests in tensor_test.cc that feed hand-crafted CSF messages through ReadSparseTensor. You're right about ndim=1: it has no indptr buffers, so it slipped past the old `ndim < 1` guard and reached SparseCSFIndex's constructor, which dereferences `indptr_.front()` on an empty vector. I've tightened the reader to require ndim >= 2. The new test covers that case plus too-many indices buffers and an axisOrder/indicesBuffers length mismatch, and the existing round-trip tests still pass. -- 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]
