Skip to content

[SPARK-8784] [SQL] Add Python API for hex and unhex#7223

Closed
davies wants to merge 9 commits into
apache:masterfrom
davies:hex
Closed

[SPARK-8784] [SQL] Add Python API for hex and unhex#7223
davies wants to merge 9 commits into
apache:masterfrom
davies:hex

Conversation

@davies

@davies davies commented Jul 4, 2015

Copy link
Copy Markdown
Contributor

Add Python API for hex/unhex, also cleanup Hex/Unhex

Davies Liu added 7 commits July 2, 2015 00:03
Add Python API for hex and unhex
1a24082
address commments
c3af78c
address comments and fix test
25156b7
Update math.scala
b31fc9a
Merge branch 'master' of github.com:apache/spark into hex
49e325f 
Conflicts:
	sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
	sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala
Merge branch 'hex' of github.com:davies/spark into hex
f032fbb
Merge branch 'master' of github.com:apache/spark into hex
f9fe5a3 
Conflicts:
	sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
	sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
	sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala
@AmplabJenkins

AmplabJenkins commented Jul 4, 2015

Copy link
Copy Markdown

Merged build triggered.

@AmplabJenkins

AmplabJenkins commented Jul 4, 2015

Copy link
Copy Markdown

Merged build started.

@SparkQA

SparkQA commented Jul 4, 2015

Copy link
Copy Markdown

Test build #36535 has started for PR 7223 at commit f9fe5a3.

@SparkQA

SparkQA commented Jul 4, 2015

Copy link
Copy Markdown

Test build #36535 has finished for PR 7223 at commit f9fe5a3.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • 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)

@AmplabJenkins

AmplabJenkins commented Jul 4, 2015

Copy link
Copy Markdown

Merged build finished. Test FAILed.

@davies

davies commented Jul 4, 2015

Copy link
Copy Markdown
Contributor Author

cc @rxin

@AmplabJenkins

AmplabJenkins commented Jul 4, 2015

Copy link
Copy Markdown

Merged build triggered.

@AmplabJenkins

AmplabJenkins commented Jul 4, 2015

Copy link
Copy Markdown

Merged build started.

@SparkQA

SparkQA commented Jul 4, 2015

Copy link
Copy Markdown

Test build #36536 has started for PR 7223 at commit 711a6ed.

@SparkQA

SparkQA commented Jul 4, 2015

Copy link
Copy Markdown

Test build #36536 has finished for PR 7223 at commit 711a6ed.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • 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)

@AmplabJenkins

AmplabJenkins commented Jul 4, 2015

Copy link
Copy Markdown

Merged build finished. Test FAILed.

@AmplabJenkins

AmplabJenkins commented Jul 6, 2015

Copy link
Copy Markdown

Merged build triggered.

@AmplabJenkins

AmplabJenkins commented Jul 6, 2015

Copy link
Copy Markdown

Merged build started.

@SparkQA

SparkQA commented Jul 6, 2015

Copy link
Copy Markdown

Test build #36582 has started for PR 7223 at commit 6f1249d.

@SparkQA

SparkQA commented Jul 6, 2015

Copy link
Copy Markdown

Test build #36582 has finished for PR 7223 at commit 6f1249d.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Unhex(child: Expression) extends UnaryExpression with ExpectsInputTypes

@AmplabJenkins

AmplabJenkins commented Jul 6, 2015

Copy link
Copy Markdown

Merged build finished. Test PASSed.

@rxin

rxin commented Jul 6, 2015

Copy link
Copy Markdown
Contributor

LGTM.

@rxin

rxin commented Jul 6, 2015

Copy link
Copy Markdown
Contributor

(although asf infra is down so we cannot merge anything)

@asfgit asfgit closed this in 37e4d92 Jul 6, 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 @AmplabJenkins @SparkQA @rxin