Skip to content

[SPARK-5038][SQL] Add explicit return type for implicit functions in Spark SQL #3859

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
Closed
wants to merge 1 commit into from
Closed

[SPARK-5038][SQL] Add explicit return type for implicit functions in Spark SQL #3859

wants to merge 1 commit into from

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Dec 31, 2014

As we learned in #3580, not explicitly typing implicit functions can lead to compiler bugs and potentially unexpected runtime behavior.

@SparkQA
Copy link

SparkQA commented Dec 31, 2014

Test build #24966 has started for PR 3859 at commit 30c2c24.

  • This patch merges cleanly.

rxin added a commit to rxin/spark that referenced this pull request Dec 31, 2014
@rxin
[SPARK-5038] Add explicit return type for implicit functions.
73702f9 
This is a follow up PR for rest of Spark (outside Spark SQL).

The original PR for Spark SQL can be found at apache#3859
@rxin rxin mentioned this pull request Dec 31, 2014
Closed
@SparkQA
Copy link

SparkQA commented Dec 31, 2014

Test build #24966 has finished for PR 3859 at commit 30c2c24.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

AmplabJenkins commented Dec 31, 2014

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

@marmbrus
Copy link
Contributor

marmbrus commented Dec 31, 2014

LGTM

@rxin
Copy link
Contributor Author

rxin commented Dec 31, 2014

Merging in master. Thanks.

@asfgit asfgit closed this in c88a3d7 Dec 31, 2014
@rxin rxin deleted the sql-implicits branch January 1, 2015 00:42
asfgit pushed a commit that referenced this pull request Jan 1, 2015
@rxin
[SPARK-5038] Add explicit return type for implicit functions.
7749dd6 
As we learned in #3580, not explicitly typing implicit functions can lead to compiler bugs and potentially unexpected runtime behavior.

This is a follow up PR for rest of Spark (outside Spark SQL). The original PR for Spark SQL can be found at #3859

Author: Reynold Xin <[email protected]>

Closes #3860 from rxin/implicit and squashes the following commits:

73702f9 [Reynold Xin] [SPARK-5038] Add explicit return type for implicit functions.
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

@rxin @SparkQA @AmplabJenkins @marmbrus