-
Notifications
You must be signed in to change notification settings - Fork 9
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
Struct #9
Comments
Great, thanks! I don't know what you have planned, and I haven't really given this any thought yet, but one thing I'd like to recommend is that you consider that named tuples might be useful. These will be a core part of the language starting with 0.7. I don't actually know for sure whether they are relevant here, but it's a thought. |
I was thinking more along the lines of any Julia struct. Though I haven’t fully considered how that would work exactly. |
Definitely yes, we'd ultimately want the ability to store any Julia struct. One thing to keep in mind though: to get good performance you are going to have to make sure that you are not doing anything that triggers "extra" compilation. By this I mean calling a function with values that can only be determined at run-time that calls a macro that defines a Julia |
Hi - how much work do you think needs to be done in order to read arrow structs? e.g. a typical "dataframe" struct with each entry's values being a vector of the same length. |
@kcajf sorry I had missed your comment all that time ago. It's a little hard for me to guess how much effort that would be right now. I'm probably going to look into IPC first, and I think maybe implementin structs will be a necessary part of that. |
No problem. Reading the streaming IPC is actually what I was trying to do above - I've since read up an arrow enough to know the right terminology! An aside: there is an issue on this project discussing plans for merging into the official apache project. Has there been any further discussion on other channels about whether / how that would happen? |
I would like to take a shot at this one if you don’t mind. Just opening an issue to declare my intent to start on this hopefully in the next few days.
The text was updated successfully, but these errors were encountered: