luhenry commented on pull request #30810: URL: https://github.com/apache/spark/pull/30810#issuecomment-819113258
> This also changes in a few cases where native vs Java BLAS is invoked, but I think it's probably a good rationalization of those calls. I couldn't find such a case. Changes from `blas` to `BLAS.nativeBLAS` should lead to using native libraries just the same. It's only in the case that the native library is _not_ available, then it will try to fallback to VectorizedBLAS instead of F2jBLAS. > If you have a moment, you might also do one last check through the source for other occurrences of BLAS that may have arisen after you began the change. @zhengruifeng has been applying BLAS in more places. I couldn't find any other occurrence. Thank you again :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
