Skip to content

[SPARK-9500] add TernaryExpression to simplify ternary expressions#7816

Closed
davies wants to merge 1 commit intoapache:masterfrom
davies:ternary
Closed

[SPARK-9500] add TernaryExpression to simplify ternary expressions#7816
davies wants to merge 1 commit intoapache:masterfrom
davies:ternary

Conversation

@davies
Copy link
Copy Markdown
Contributor

@davies davies commented Jul 31, 2015

There lots of duplicated code in ternary expressions, create a TernaryExpression for them to reduce duplicated code.

cc @chenghao-intel

@davies davies mentioned this pull request Jul 31, 2015
Closed
@SparkQA
Copy link
Copy Markdown

SparkQA commented Jul 31, 2015

Test build #39170 has finished for PR 7816 at commit ed2bf76.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • abstract class TernaryExpression extends Expression

@chenghao-intel
Copy link
Copy Markdown
Contributor

chenghao-intel commented Jul 31, 2015

LGTM

1 similar comment
@cloud-fan
Copy link
Copy Markdown
Contributor

cloud-fan commented Jul 31, 2015

LGTM

@asfgit asfgit closed this in 6bba750 Jul 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@davies @SparkQA @chenghao-intel @cloud-fan