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
[SPARK-37534][BUILD][3.2] Bump dev.ludovic.netlib to 2.2.1 #34797
Conversation
ok to test |
@luhenry mind filling other sections in the PR description? |
2.2.1 fixes a bug in sgemm
@HyukjinKwon I updated it. Is it enough details, or should I add some more? |
👍 |
d723b83
to
eb6f8cc
Compare
Kubernetes integration test starting |
Kubernetes integration test status failure |
Test build #145899 has finished for PR 34797 at commit
|
Can one of the admins verify this patch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, LGTM. Thank you, @luhenry and @HyukjinKwon .
Merged to branch-3.2 for Apache Spark 3.2.1.
### What changes were proposed in this pull request? Bump the version of dev.ludovic.netlib from 2.2.0 to 2.2.1. ### Why are the changes needed? This fixes a computation bug in sgemm. See [the issue](luhenry/netlib#7) and [the diff](luhenry/netlib@v2.2.0...v2.2.1). ### Does this PR introduce _any_ user-facing change? No. It fixes the underlying computation of a single floating point matrix-matrix multiplication. ### How was this patch tested? Additional tests were added to the dev.ludovic.netlib project, and a CI run has validated the change. Closes #34797 from luhenry/branch-3.2-bump-netlib. Authored-by: Ludovic Henry <git@ludovic.dev> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
### What changes were proposed in this pull request? Bump the version of dev.ludovic.netlib from 2.2.0 to 2.2.1. ### Why are the changes needed? This fixes a computation bug in sgemm. See [the issue](luhenry/netlib#7) and [the diff](luhenry/netlib@v2.2.0...v2.2.1). ### Does this PR introduce _any_ user-facing change? No. It fixes the underlying computation of a single floating point matrix-matrix multiplication. ### How was this patch tested? Additional tests were added to the dev.ludovic.netlib project, and a CI run has validated the change. Closes apache#34797 from luhenry/branch-3.2-bump-netlib. Authored-by: Ludovic Henry <git@ludovic.dev> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
### What changes were proposed in this pull request? Bump the version of dev.ludovic.netlib from 2.2.0 to 2.2.1. ### Why are the changes needed? This fixes a computation bug in sgemm. See [the issue](luhenry/netlib#7) and [the diff](luhenry/netlib@v2.2.0...v2.2.1). ### Does this PR introduce _any_ user-facing change? No. It fixes the underlying computation of a single floating point matrix-matrix multiplication. ### How was this patch tested? Additional tests were added to the dev.ludovic.netlib project, and a CI run has validated the change. Closes apache#34797 from luhenry/branch-3.2-bump-netlib. Authored-by: Ludovic Henry <git@ludovic.dev> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
### What changes were proposed in this pull request? Bump the version of dev.ludovic.netlib from 2.2.0 to 2.2.1. ### Why are the changes needed? This fixes a computation bug in sgemm. See [the issue](luhenry/netlib#7) and [the diff](luhenry/netlib@v2.2.0...v2.2.1). ### Does this PR introduce _any_ user-facing change? No. It fixes the underlying computation of a single floating point matrix-matrix multiplication. ### How was this patch tested? Additional tests were added to the dev.ludovic.netlib project, and a CI run has validated the change. Closes apache#34797 from luhenry/branch-3.2-bump-netlib. Authored-by: Ludovic Henry <git@ludovic.dev> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
What changes were proposed in this pull request?
Bump the version of dev.ludovic.netlib from 2.2.0 to 2.2.1.
Why are the changes needed?
This fixes a computation bug in sgemm. See the issue and the diff.
Does this PR introduce any user-facing change?
No. It fixes the underlying computation of a single floating point matrix-matrix multiplication.
How was this patch tested?
Additional tests were added to the dev.ludovic.netlib project, and a CI run has validated the change.