-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: nanoarrow refactor #45
Conversation
Do you have plans to continue working on this feature and merge it eventually? Or was it just an experiment? |
Which part? The nanoarrow refactor or writing to wasm memory? This PR is kinda intertwined among the two. The nanoarrow refactor is less likely to get merged; the main goal of that is to remove an Arrow JS dependency, but I don't think I have the development bandwidth to maintain a separate (even extremely minimal) Arrow implementation, so I'll probably shelve that. The writing part is useful even with Arrow JS and I'd like to have that merged, though I don't have a use case for it myself yet. |
If there's any of this that you'd like to work on, feel free to discuss more and/or make a PR |
I'm going to close this because I don't have the maintenance bandwidth to try and implement data structures for Arrow outside of Arrow JS, and I don't have a use case at this point where Arrow JS's bundle size is a deal-breaker. |
Closes #41, closes #42
Change list:
malloc
function exported from wasm.Todo: