Skip to content
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

[SPARK-3814][SQL] Support for Bitwise AND(&), OR(|) ,XOR(^), NOT(~) in Spark HQL and SQL #2961

Closed
wants to merge 1 commit into from

Conversation

ravipesala
Copy link
Contributor

Currently there is no support of Bitwise & , | in Spark HiveQl and Spark SQL as well. So this PR support the same.
I am closing #2926 as it has conflicts to merge. And also added support for Bitwise AND(&), OR(|) ,XOR(^), NOT(~) And I handled all review comments in that PR

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@ravipesala
Copy link
Contributor Author

test this please

@marmbrus
Copy link
Contributor

ok to test

@SparkQA
Copy link

SparkQA commented Oct 28, 2014

Test build #22329 has started for PR 2961 at commit a391c7a.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 28, 2014

Test build #22329 has finished for PR 2961 at commit a391c7a.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class BitwiseAnd(left: Expression, right: Expression) extends BinaryArithmetic
    • case class BitwiseOr(left: Expression, right: Expression) extends BinaryArithmetic
    • case class BitwiseXor(left: Expression, right: Expression) extends BinaryArithmetic
    • case class BitwiseNot(child: Expression) extends UnaryExpression

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/22329/
Test PASSed.

@marmbrus
Copy link
Contributor

Thanks for implementing this! Merged to master.

@asfgit asfgit closed this in 5807cb4 Oct 28, 2014
@ravipesala ravipesala deleted the SPARK-3814-NEW4 branch October 29, 2014 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants