Skip to content

Conversation

@luhenry
Copy link
Contributor

@luhenry luhenry commented Dec 2, 2021

What changes were proposed in this pull request?

Bump the version of dev.ludovic.netlib from 2.2.0 to 2.2.1. This fixes a computation bug in sgemm. See [1]. Diff is [2]

[1] luhenry/netlib#7
[2] luhenry/netlib@v2.2.0...v2.2.1

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@github-actions github-actions bot added the BUILD label Dec 2, 2021
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

2.2.1 fixes a bug in sgemm
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for making a PR, @luhenry .
Do you think you can file an Apache Spark JIRA issue for this and use the JIRA ID prefix in your PR title?

@luhenry luhenry changed the title Bump dev.ludovic.netlib to 2.2.1 [SPARK-37534] Bump dev.ludovic.netlib to 2.2.1 Dec 3, 2021
@luhenry
Copy link
Contributor Author

luhenry commented Dec 3, 2021

@dongjoon-hyun I created the issue at https://issues.apache.org/jira/browse/SPARK-37534 and linked it in the title.

@dongjoon-hyun
Copy link
Member

Thank you, @luhenry .

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-37534] Bump dev.ludovic.netlib to 2.2.1 [SPARK-37534][BUILD] Bump dev.ludovic.netlib to 2.2.1 Dec 3, 2021
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Merged to master.

@dongjoon-hyun
Copy link
Member

dongjoon-hyun commented Dec 3, 2021

According to the context, I switched your JIRA type from Improvement to Bug in order to allow backporting. Could you make a backporting PR to branch-3.2, @luhenry ? We need to pass CI once more on that branch.

@luhenry
Copy link
Contributor Author

luhenry commented Dec 3, 2021

@dongjoon-hyun thank you for merging! I submitted #34797 for 3.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants