maropu commented on a change in pull request #31623:
URL: https://github.com/apache/spark/pull/31623#discussion_r581596104
##########
File path: core/src/main/scala/org/apache/spark/util/DependencyUtils.scala
##########
@@ -59,8 +59,8 @@ private[spark] object DependencyUtils extends Logging {
* @param uri Ivy URI need to be downloaded.
* @return Tuple value of parameter `transitive` and `exclude` value.
*
- * 1. transitive: whether to download dependency jar of Ivy URI,
default value is false
- * and this parameter value is case-sensitive. Invalid value will
be treat as false.
+ * 1. transitive: whether to download dependency jar of Ivy URI,
default value is true
+ * and this parameter value is case-insensitive. Invalid value
will be treat as false.
Review comment:
Could you leave some comments like `this behaviour follows hive one...`?
----------------------------------------------------------------
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]
