-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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-13789: [Go] Implement Scalar Values for Go #11024
Conversation
bump @emkornfield @sbinet |
Sorry didn't finish reviewing, will try to finish review over the next week or two. |
@emkornfield any luck? |
} | ||
|
||
if !l.Valid { | ||
return |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you want to set an err in this case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that case isn't an error, it just means it's a null scalar
@emkornfield do you have any objections to me merging this? |
sorry have not had a chance to look yet. |
This is needed for adding nested types to the `MakeArrayFromScalar` function in ARROW-13789 (#11024) @emkornfield @sbinet @fsaintjacques Closes #11128 from zeroshade/concat_arrays Lead-authored-by: Matthew Topol <mtopol@factset.com> Co-authored-by: Matt Topol <mtopol@factset.com> Signed-off-by: Matthew Topol <mtopol@factset.com>
@emkornfield Do you have any objections to me merging this for now so i can continue the development and it can be further reviewed later as I add more things? I'm still trying to keep the PRs smaller where possible so i don't want to just add more stuff to this PR |
sorry, yes, I think merging is fine, I'll try to get some time to review. |
This is needed for adding nested types to the `MakeArrayFromScalar` function in ARROW-13789 (apache#11024) @emkornfield @sbinet @fsaintjacques Closes apache#11128 from zeroshade/concat_arrays Lead-authored-by: Matthew Topol <mtopol@factset.com> Co-authored-by: Matt Topol <mtopol@factset.com> Signed-off-by: Matthew Topol <mtopol@factset.com>
In preparation for trying to expose the Compute APIs vs CGO, first we need to have Scalar values implemented. After this, we can implement compatible Expression tree handling for serializing between the C++ and the Golang Closes apache#11024 from zeroshade/scalars Lead-authored-by: Matthew Topol <mtopol@factset.com> Co-authored-by: Matt Topol <mtopol@factset.com> Signed-off-by: Matthew Topol <mtopol@factset.com>
In preparation for trying to expose the Compute APIs vs CGO, first we need to have Scalar values implemented. After this, we can implement compatible Expression tree handling for serializing between the C++ and the Golang