paleolimbot commented on PR #397: URL: https://github.com/apache/arrow-nanoarrow/pull/397#issuecomment-1979033191
> I did leave some larger comments this time around. Great points! I think one of the current barriers is that there is no object type on which one could implement `to_string()` (or in this case, `write_string_into()` so that some writable object has an opportunity to signal that no further input needs to be written). A quasi-barrier is that it is "expensive" to instantiate the iterator, although in the context of printing things that is probably not important. I think the solution is to have a `Scalar` class (composed of a `CArray` and an offset) instead of (mis)using the iterator infrastructure in a hard-to understand way. I'll add this to the PR implementing `nanoarrow.Array` and circle back here! -- 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]
