Closed wajda closed 1 year ago
Thanks @Zejnilovic for reporting it
The compile scope dependency tree for bundle-3.3:
and for the bundle-2.4 for comparison:
The only spotted difference - the commons-configuration:1.6
was missing in 2.4 but present in 3.3 due to the difference in Spark provided environment. This dependency is a part of Spline public API (appears in Plugin API), so it has to stay unshaded anyway.
The graph-core
contains packages scala.collection
and scalax
which makes it dangerous to relocate due to too broad pattern.
The absa-shaded
is already shaded under za.co.absa.shaded
The rest dependencies are to be relocated to za.co.absa.spline.shaded
As a result of a further discussion with Adam we decided to leave scalaj.http
library unshaded, The reasons were:
Scalaj.http
is among of them.Scalaj.http
has been stable for a long time and is now deprecated, so we expect really small collision probability on that library.We also discussed another option - in addition to every agent bundle jar produce another jar with the classifier shaded
(e.g spark-3.2-spline-agent-bundle_2.12-1.2.1-shaded.jar
). The difference would be that the original jar would have dependencies unshaded as before, but the new one would have them shaded. This would allow the users to decide if they want deps shaded or not depending on their use-case. The downside of this approach is that for majority of users it would be difficult to make a decision which jar to use, or their use-case might change, so the decision they make could be often wrong. So having two variants of jar publicly available could quickly lead to even more confusion around managing Spline dependencies. So in the end, we decided not to go down that route.