Skip to content

Conversation

@squito
Copy link
Contributor

@squito squito commented Dec 15, 2016

What changes were proposed in this pull request?

93cdb8a Introduced a compile error under scala 2.10, this fixes that error.

How was this patch tested?

locally ran

dev/change-version-to-2.10.sh
build/sbt -Pyarn -Phadoop-2.4 -Dhadoop.version=2.6.0 -Dscala-2.10 "project yarn" "test-only *YarnAllocatorSuite"

(which failed at test compilation before this change)

@kayousterhout
Copy link
Contributor

LGTM

@SparkQA
Copy link

SparkQA commented Dec 15, 2016

Test build #70215 has finished for PR 16298 at commit e25dbfe.

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

@rxin
Copy link
Contributor

rxin commented Dec 15, 2016

Merging in master.

@asfgit asfgit closed this in 32ff964 Dec 15, 2016
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
## What changes were proposed in this pull request?

apache@93cdb8a Introduced a compile error under scala 2.10, this fixes that error.

## How was this patch tested?

locally ran
```
dev/change-version-to-2.10.sh
build/sbt -Pyarn -Phadoop-2.4 -Dhadoop.version=2.6.0 -Dscala-2.10 "project yarn" "test-only *YarnAllocatorSuite"
```
(which failed at test compilation before this change)

Author: Imran Rashid <irashid@cloudera.com>

Closes apache#16298 from squito/blacklist-2.10.
yoonlee95 pushed a commit to yoonlee95/spark that referenced this pull request Aug 17, 2017
## What changes were proposed in this pull request?

apache@93cdb8a Introduced a compile error under scala 2.10, this fixes that error.

## How was this patch tested?

locally ran
```
dev/change-version-to-2.10.sh
build/sbt -Pyarn -Phadoop-2.4 -Dhadoop.version=2.6.0 -Dscala-2.10 "project yarn" "test-only *YarnAllocatorSuite"
```
(which failed at test compilation before this change)

Author: Imran Rashid <irashid@cloudera.com>

Closes apache#16298 from squito/blacklist-2.10.
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