-
Notifications
You must be signed in to change notification settings - Fork 15
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
bug: Figure out plan for supporting Datums / new Arrow APIs #783
Comments
At a high level, this is nice -- it makes it easier to work with Literals. We currently intend to distinguish between Generally, we think is still likely the best path, from the terms of simplicity of executing the plans and supporting instructions like Alternative:
This could open the possibility of having a |
This updates many places to use the new `Datum` backed APIs. But, it suppresses a few cases and references #783. Specifically, some expression code needs to be figured out, some runtime code (likely to be deleted) and some other places.
This updates many places to use the new `Datum` backed APIs. But, it suppresses a few cases and references #783. Specifically, some expression code needs to be figured out, some runtime code (likely to be deleted) and some other places. Seems to fix the behavior of time-delta comparisons to literals (and possibly other logical types represented by numeric primitives).
Description
Arrow is introducing a
Datum
type which makes it easier to support arrays and scalar values in a variety of methods. They are gradually implementing new kernels based on theDatum
type and deprecating the old methods. We should figure out what this means for our expression evaluators.The text was updated successfully, but these errors were encountered: