vmalakhin commented on pull request #34383: URL: https://github.com/apache/spark/pull/34383#issuecomment-952104812
> codehaus jackson is 1.x, so is more "OK" to add back. That said it's probably excluded because otherwise unused, and triggers security warnings on static analysis, so probably why it wasn't excluded. > > fasterxml jackson is probably specifically excluded because it is included at a newer version in the Spark build. That isn't related to the error you show. Neither is Guava, which is in a similar situation. Those shouldn't be changed. > > Can you just add the dependencies that ABFS requires to your app? I don't think this profile is meant to support third party libraries, though ABFS connector could be arguably a special case. > > hadoop-cloud isn't published as part of the binary release so is more "OK" to change in this way though. All in all I could see adding back codehaus jackson Yep, the only difference is just asl jar in the output. If it's OK to go ahead can we merge this in? Or please let me know next steps. Thanks! -- 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]
