You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by we...@apache.org on 2021/01/21 12:43:14 UTC

[spark] branch branch-3.1 updated: [SPARK-33901][SQL][FOLLOWUP] Add drop table in charvarchar test

This is an automated email from the ASF dual-hosted git repository.

wenchen pushed a commit to branch branch-3.1
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.1 by this push:
     new acb803b  [SPARK-33901][SQL][FOLLOWUP] Add drop table in charvarchar test
acb803b is described below

commit acb803b440b837457da809aa2f694a2c73ac1d2f
Author: ulysses-you <ul...@gmail.com>
AuthorDate: Thu Jan 21 12:41:52 2021 +0000

    [SPARK-33901][SQL][FOLLOWUP] Add drop table in charvarchar test
    
    ### What changes were proposed in this pull request?
    
    Add `drop table` in charvarchar sql test.
    
    ### Why are the changes needed?
    
    1. `drop table` is also a test case, for better coverage.
    2. It's more clear to drop table which created in current test.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Add test.
    
    Closes #31277 from ulysses-you/SPARK-33901-FOLLOWUP.
    
    Authored-by: ulysses-you <ul...@gmail.com>
    Signed-off-by: Wenchen Fan <we...@databricks.com>
    (cherry picked from commit da4b50f8e2f7299c9e6f8bafb541c9f785938e4a)
    Signed-off-by: Wenchen Fan <we...@databricks.com>
---
 .../resources/sql-tests/inputs/charvarchar.sql     |  5 +++++
 .../sql-tests/results/charvarchar.sql.out          | 26 +++++++++++++++++++++-
 2 files changed, 30 insertions(+), 1 deletion(-)

diff --git a/sql/core/src/test/resources/sql-tests/inputs/charvarchar.sql b/sql/core/src/test/resources/sql-tests/inputs/charvarchar.sql
index dbdb8cc..b49920e 100644
--- a/sql/core/src/test/resources/sql-tests/inputs/charvarchar.sql
+++ b/sql/core/src/test/resources/sql-tests/inputs/charvarchar.sql
@@ -60,3 +60,8 @@ desc formatted char_part;
 
 MSCK REPAIR TABLE char_part;
 desc formatted char_part;
+
+-- char_tbl has renamed to char_tbl1
+drop table char_tbl1;
+drop table char_tbl2;
+drop table char_tbl3;
diff --git a/sql/core/src/test/resources/sql-tests/results/charvarchar.sql.out b/sql/core/src/test/resources/sql-tests/results/charvarchar.sql.out
index 774235e..a4d0b55 100644
--- a/sql/core/src/test/resources/sql-tests/results/charvarchar.sql.out
+++ b/sql/core/src/test/resources/sql-tests/results/charvarchar.sql.out
@@ -1,5 +1,5 @@
 -- Automatically generated by SQLQueryTestSuite
--- Number of queries: 45
+-- Number of queries: 48
 
 
 -- !query
@@ -661,3 +661,27 @@ Type                	MANAGED
 Provider            	parquet             	                    
 Location [not included in comparison]/{warehouse_dir}/char_part	                    
 Partition Provider  	Catalog
+
+
+-- !query
+drop table char_tbl1
+-- !query schema
+struct<>
+-- !query output
+
+
+
+-- !query
+drop table char_tbl2
+-- !query schema
+struct<>
+-- !query output
+
+
+
+-- !query
+drop table char_tbl3
+-- !query schema
+struct<>
+-- !query output
+


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org