andishgar commented on issue #47613: URL: https://github.com/apache/arrow/issues/47613#issuecomment-3317912631
I mean that the same integer type is assumed for every `indptr` buffer in the C++ IPC implementation, but in reality, each `indptr` buffer can have its own integer type. For example, one `indptr` buffer could be `int32`, while another could be `int64`. However, in the C++ IPC implementation, only a single integer type is considered for all `indptr` buffers: C++implementation https://github.com/apache/arrow/blob/d803afcc43f5d132506318fd9e162d33b2c3d4cd/cpp/src/arrow/ipc/metadata_internal.cc#L1169-L1175 This happens because only one type is maintained for all [indptr buffers](https://github.com/apache/arrow/blob/d803afcc43f5d132506318fd9e162d33b2c3d4cd/format/SparseTensor.fbs#L160). However, each buffer actually has its own type. The same issue also exists for the `indices` buffers. -- 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]
