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

[BACKPORT-2.1][SPARK-19999][core] Workaround JDK-8165231 to identify PPC64 architectures as supporting unaligned access #17509

Closed
wants to merge 1 commit into from

Conversation

kiszk
Copy link
Member

@kiszk kiszk commented Apr 2, 2017

What changes were proposed in this pull request?

This PR is backport of #17472 to Spark 2.1

java.nio.Bits.unaligned() does not return true for the ppc64le arch.
see https://bugs.openjdk.java.net/browse/JDK-8165231
Check architecture in Platform.java

How was this patch tested?

unit test

@SparkQA
Copy link

SparkQA commented Apr 2, 2017

Test build #75457 has finished for PR 17509 at commit 5c93197.

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

@srowen
Copy link
Member

srowen commented Apr 2, 2017

Merged to 2.1

asfgit pushed a commit that referenced this pull request Apr 2, 2017
…PPC64 architectures as supporting unaligned access

## What changes were proposed in this pull request?

This PR is backport of #17472 to Spark 2.1

java.nio.Bits.unaligned() does not return true for the ppc64le arch.
see [https://bugs.openjdk.java.net/browse/JDK-8165231](https://bugs.openjdk.java.net/browse/JDK-8165231)
Check architecture in Platform.java

## How was this patch tested?

unit test

Author: Kazuaki Ishizaki <[email protected]>

Closes #17509 from kiszk/branch-2.1.
@maropu
Copy link
Member

maropu commented Apr 21, 2017

@kiszk Cloud you close this?

@kiszk
Copy link
Member Author

kiszk commented Apr 21, 2017

Sure, thanks

@kiszk kiszk closed this Apr 21, 2017
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