Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19823#discussion_r153091293
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -341,6 +341,12 @@ case class LoadDataCommand(
} else {
val uri = new URI(path)
if (uri.getScheme() != null && uri.getAuthority() != null) {
+ val hadoopConf = sparkSession.sessionState.newHadoopConf()
+ val srcPath = new Path(path)
--- End diff --
nit: Let's use `new Path(uri)`. I think we better validate `uri` in this
scope.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
