Skip to content

ARROW-9988: [Rust] [DataFusion] Added +-/* as operators to logical expressions.#8182

Closed
jorgecarleitao wants to merge 1 commit intoapache:masterfrom
jorgecarleitao:ops
Closed

ARROW-9988: [Rust] [DataFusion] Added +-/* as operators to logical expressions.#8182
jorgecarleitao wants to merge 1 commit intoapache:masterfrom
jorgecarleitao:ops

Conversation

@jorgecarleitao
Copy link
Copy Markdown
Member

Also moved operators to their own module.

@github-actions
Copy link
Copy Markdown

Also moved operators to their own module.
Copy link
Copy Markdown
Member

@andygrove andygrove left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@andygrove andygrove closed this in b3a6da1 Sep 15, 2020
andygrove pushed a commit that referenced this pull request Sep 17, 2020
This PR is built on top of #8182

This PR fixes the operator NOT, that currently cannot be logically planned. It was also an operator that was considered a Binary, even though it accepts only a single argument.

Closes #8183 from jorgecarleitao/fix_not

Authored-by: Jorge C. Leitao <[email protected]>
Signed-off-by: Andy Grove <[email protected]>
@jorgecarleitao jorgecarleitao deleted the ops branch October 28, 2020 04:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants