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 b97834a58fb29999a0a98eb2645bd9e77e97209b
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

Reply via email to