[SPARK-19257] The type of CatalogStorageFormat.locationUri should be java.net.URI instead of String

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.2.0
• Component/s: SQL
• Target Version/s: 2.2.0
• Assignee: Song Jun
• Reporter: Wenchen Fan
• Created: 17/Jan/17 07:24
• Updated: 06/Mar/17 18:45
• Resolved: 06/Mar/17 18:44

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

Issue Links

• is duplicated by: SPARK-19332 table's location should check if a URI is legal
• links to: [Github] Pull Request #16619 (windpiger)
• links to: [Github] Pull Request #17149 (windpiger)