[ 
https://issues.apache.org/jira/browse/SPARK-19257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wenchen Fan updated SPARK-19257:
--------------------------------
    Description: 
Currently we treat `CatalogStorageFormat.locationUri` as URI string and always 
convert it to path by `new Path(new URI(locationUri))`

It will be safer if we can make the type of `CatalogStorageFormat.locationUri` 
java.net.URI. We should finish the TODO in 
https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala#L50-L52

  was:
Currently we treat `CatalogStorageFormat.locationUri` as URI string and always 
convert it to path by `new Path(new URI(locationUri))`

It will be safer if we can make the type of `CatalogStorageFormat.locationUri` 
java.net.URI.


> The type of CatalogStorageFormat.locationUri should be java.net.URI instead 
> of String
> -------------------------------------------------------------------------------------
>
>                 Key: SPARK-19257
>                 URL: https://issues.apache.org/jira/browse/SPARK-19257
>             Project: Spark
>          Issue Type: Improvement
>          Components: SQL
>            Reporter: Wenchen Fan
>
> Currently we treat `CatalogStorageFormat.locationUri` as URI string and 
> always convert it to path by `new Path(new URI(locationUri))`
> It will be safer if we can make the type of 
> `CatalogStorageFormat.locationUri` java.net.URI. We should finish the TODO in 
> https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala#L50-L52



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to