Skip to content

Conversation

@maver1ck
Copy link
Contributor

@maver1ck maver1ck commented Aug 2, 2016

What changes were proposed in this pull request?

Casting ConcurrentHashMap to ConcurrentMap allows to run code compiled with Java 8 on Java 7

How was this patch tested?

Compilation. Existing automatic tests

@maver1ck maver1ck changed the title [SPARK-15541] Casting ConcurrentHashMap to ConcurrentMap [SPARK-15541] Casting ConcurrentHashMap to ConcurrentMap (master branch) Aug 2, 2016
@SparkQA
Copy link

SparkQA commented Aug 2, 2016

Test build #63118 has finished for PR 14459 at commit 471999a.

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

asfgit pushed a commit that referenced this pull request Aug 2, 2016
## What changes were proposed in this pull request?

Casting ConcurrentHashMap to ConcurrentMap allows to run code compiled with Java 8 on Java 7

## How was this patch tested?

Compilation. Existing automatic tests

Author: Maciej Brynski <[email protected]>

Closes #14459 from maver1ck/spark-15541-master.

(cherry picked from commit 511dede)
Signed-off-by: Sean Owen <[email protected]>
@asfgit asfgit closed this in 511dede Aug 2, 2016
@srowen
Copy link
Member

srowen commented Aug 2, 2016

Merged to master/2.0/1.6

asfgit pushed a commit that referenced this pull request Aug 2, 2016
Casting ConcurrentHashMap to ConcurrentMap allows to run code compiled with Java 8 on Java 7

Compilation. Existing automatic tests

Author: Maciej Brynski <[email protected]>

Closes #14459 from maver1ck/spark-15541-master.

(cherry picked from commit 511dede)
Signed-off-by: Sean Owen <[email protected]>
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Aug 3, 2016
Casting ConcurrentHashMap to ConcurrentMap allows to run code compiled with Java 8 on Java 7

Compilation. Existing automatic tests

Author: Maciej Brynski <[email protected]>

Closes apache#14459 from maver1ck/spark-15541-master.

(cherry picked from commit 511dede)
Signed-off-by: Sean Owen <[email protected]>
(cherry picked from commit 8a22275)
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.

3 participants