paleolimbot commented on PR #17: URL: https://github.com/apache/arrow-nanoarrow/pull/17#issuecomment-1211464135
Ok - this is a first pass at #8 that implements the functions needed to make "build by buffer" a thing. The `ArrowBufferAppendInt8()` family of functions helps make code that does `switch(some_arrow_type) { ... }` prettier. Rather than make `ArrowArrayAppendInt8()`, this just exposes the internal buffer objects and lets you do all the buffer method things you need to do (requiring some knowledge of the spec). Rowwise builders can cache the arrow type and do `switch (the_arrow_type) { ... }` and append accordingly. Most of this was implemented first in #16 but it sounds like many of those changes aren't required to get started on database driver work. This PR is intended to be the minimum change needed to facilitate that work. -- 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: issues-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org