[SPARK-3967] don’t redundantly overwrite executor JAR deps

cool, I've moved to that. we rearranged the logic deck-chairs of this block during an earlier review pass and settled on the `var shouldCopy` approach, but I can accept the `return` as being marginally less aesthetically unappealing :)