Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/22927#discussion_r230217147 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala --- @@ -393,7 +393,7 @@ object LoadDataCommand { throw new IllegalArgumentException(e) } } else { - path + new Path(pathUri) --- End diff -- Here, it is converted PATH to URI and then converted back to Path. What is your goal rather than directly building a path? ``` if (path.isAbsolute()) path else new Path(workingDir, path) ````
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org