[WIP][SPARK-19257][SQL]CatalogStorageFormat.locationUri should be java.net.URI

[windpiger]

What changes were proposed in this pull request?

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

How was this patch tested?

[SparkQA]

Test build #71514 has finished for PR 16619 at commit 2ab4928.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #71515 has finished for PR 16619 at commit 758c8b5.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #71516 has finished for PR 16619 at commit 5b5debf.

• This patch fails to build.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #71518 has finished for PR 16619 at commit 635a398.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #71700 has finished for PR 16619 at commit 66dc4de.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #71766 has finished for PR 16619 at commit 59736fa.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[srowen]

Should this be closed in favor of #17149 ?

[windpiger]

yes I will close it , thanks! @srowen