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.

Sign up for GitHub

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

Jump to bottom

[SPARK-8822][SQL] clean up type checking in math.scala. #7220

Closed

rxin wants to merge 3 commits into apache:master from rxin:SPARK-8822

Contributor

rxin commented Jul 4, 2015

No description provided.


          [SPARK-8822][SQL] clean up type checking in math.scala.

beb2a97

Contributor Author

rxin commented Jul 4, 2015

rxin added 2 commits

July 3, 2015 22:54


          Fixed type precedence.

22d0463


          Test cases.

0cda076

SparkQA commented Jul 4, 2015

Test build #36524 has finished for PR 7220 at commit 0cda076.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
- case class ScalaUDF(
- case class CurrentDate() extends LeafExpression
- case class CurrentTimestamp() extends LeafExpression
- case class Hex(child: Expression) extends UnaryExpression with ExpectsInputTypes
- case class UnHex(child: Expression) extends UnaryExpression with ExpectsInputTypes
- case class ShiftLeft(left: Expression, right: Expression)
- case class ShiftRight(left: Expression, right: Expression)
- case class ShiftRightUnsigned(left: Expression, right: Expression)
- case class UserDefinedFunction protected[sql] (

Contributor

davies commented Jul 4, 2015

LGTM

Contributor Author

rxin commented Jul 4, 2015

Thanks - merged.

Contributor Author

rxin commented Jul 4, 2015

(just realized this conflicts with your patch that I reverted - but hopefully it's easy to resolve the conflicts)

asfgit closed this in

c991ef5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment