Skip to content

[SPARK-20554][BUILD] Remove usage of scala.language.reflectiveCalls - #17949

Closed
srowen wants to merge 1 commit into
apache:masterfrom
srowen:SPARK-20554
Closed

[SPARK-20554][BUILD] Remove usage of scala.language.reflectiveCalls#17949
srowen wants to merge 1 commit into
apache:masterfrom
srowen:SPARK-20554

Conversation

@srowen

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Remove uses of scala.language.reflectiveCalls that are either unnecessary or probably resulting in more complex code. This turned out to be less significant than I thought, but, still worth a touch-up.

How was this patch tested?

Existing tests.

…sary or probably resulting in more complex code
@SparkQA

Copy link
Copy Markdown

Test build #76798 has finished for PR 17949 at commit 104b604.

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

@srowen

Copy link
Copy Markdown
MemberAuthor

Merged to master/2.2

asfgit pushed a commit that referenced this pull request May 12, 2017
## What changes were proposed in this pull request?
Remove uses of scala.language.reflectiveCalls that are either unnecessary or probably resulting in more complex code. This turned out to be less significant than I thought, but, still worth a touch-up.
## How was this patch tested?
Existing tests.
Author: Sean Owen <sowen@cloudera.com>
Closes#17949 from srowen/SPARK-20554.
(cherry picked from commit fc8a2b6)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@srowen
srowen deleted the SPARK-20554 branch May 15, 2017 06:50
robert3005 pushed a commit to palantir/spark that referenced this pull request May 19, 2017
## What changes were proposed in this pull request?
Remove uses of scala.language.reflectiveCalls that are either unnecessary or probably resulting in more complex code. This turned out to be less significant than I thought, but, still worth a touch-up.
## How was this patch tested?
Existing tests.
Author: Sean Owen <sowen@cloudera.com>
Closesapache#17949 from srowen/SPARK-20554.
lycplus pushed a commit to lycplus/spark that referenced this pull request May 24, 2017
## What changes were proposed in this pull request?
Remove uses of scala.language.reflectiveCalls that are either unnecessary or probably resulting in more complex code. This turned out to be less significant than I thought, but, still worth a touch-up.
## How was this patch tested?
Existing tests.
Author: Sean Owen <sowen@cloudera.com>
Closesapache#17949 from srowen/SPARK-20554.
Sign up for freeto 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.

2 participants

@srowen@SparkQA