-
Notifications
You must be signed in to change notification settings - Fork 64
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
Arrow type bindings #173
Arrow type bindings #173
Conversation
LGTM overall, but can we return |
@andygrove just saw this. Yes. Question, what kind of error do we want to thrown when something happens like this you think? Should we add one for "UnImplemented"? |
Yes, in the core DataFusion crate we have |
CI is failing due to rate limits trying to consume the |
|
Which issue does this PR close?
Closes #172