paleolimbot commented on issue #27499: URL: https://github.com/apache/arrow/issues/27499#issuecomment-1494815207
I think the prototype you're referring to is here: https://gist.github.com/paleolimbot/c42f068c2b8b98255dbfbe379d905607 This prototype is for the tensor extension type, which is more like a tensor as a column in a table and is, I believe the preferred way to deal with tensors since the extension type works with the C data interface and ArrowTensor does not (as I understand it, ArrowTensor is a C++ class and an item in the IPC spec but has little support elsewhere). The extension type will have C++ support so it should be straightforward to add a wrapper (i.e., the prototype I did probably won't be necessary). It should work like any other parameterized type (i.e., you could use something like the wrapper around `ListType` as an example for how to implement it). I don't remember if that PR has merged yet. We could also wrap the `ArrowTensor` C++ class, which is I believe what you're referring to. I'm a little concerned that adding support for that particular class is adding support for something that will be deprecated soon and hasn't seen much use elsewhere. -- 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]
