pan3793 commented on PR #52653: URL: https://github.com/apache/spark/pull/52653#issuecomment-3421749691
> I had already seen the thread and I understand this solution is just a workaround but I think it's better to keep up with the latest SBT if workaround exists. @sarutak, I agree to move forward. > Anyway, `mllib-local` transitively depends on slf4j-2.0.17 when building with Maven so it's better to override or exclude slf4j-1.7.5 when building with SBT for the consistency. Excluding `slf4j-api` from `mllib-local` looks dangerous to me, because it's likely that the invoked code from deps calls `slf4j-api` underlying. I have tested to only do overriding, `dev/mima` works. Can you revert the `slf4j-api` exclusion from `mllib-local`? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
