GitHub user xubo245 opened a pull request:
https://github.com/apache/spark/pull/20227
[SPARK-23035] Fix warning: TEMPORARY TABLE ... USING ... is deprecated and
use TempViewAlreadyExistsException when create temp view
Fix warning: TEMPORARY TABLE ... USING ... is deprecated and use
TempViewAlreadyExistsException when create temp view
There are warning when run test: test("rename temporary view - destination
table with database name")
Another problem, it throw TempTableAlreadyExistsException and output
"Temporary table '$table' already exists" when we create temp view by using
org.apache.spark.sql.catalyst.catalog.GlobalTempViewManager#create, it's
improper.
## What changes were proposed in this pull request?
Fix some warning by changing "TEMPORARY TABLE ... USING ... " to "TEMPORARY
VIEW ... USING
... "
Fix improper information about TempTableAlreadyExistsException when create
temp view
## How was this patch tested?
use old test cases, such as " test("create temporary view using") "
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/xubo245/spark fixDeprecated
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/20227.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #20227
commit b97834a58fb2a0a98eb2645bd9e77e97209b
Author: xubo245 <601450868@...>
Date: 2018-01-11T01:58:48Z
[SPARK-23035] Fix warning: TEMPORARY TABLE ... USING ... is deprecated and
use TempViewAlreadyExistsException when create temp view
Fix warning: TEMPORARY TABLE ... USING ... is deprecated and use
TempViewAlreadyExistsException when create temp view
There are warning when run test: test("rename temporary view - destination
table with database name")
Another problem, it throw TempTableAlreadyExistsException and output
"Temporary table '$table' already exists" when we create temp view by using
org.apache.spark.sql.catalyst.catalog.GlobalTempViewManager#create, it's
improper.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org