Github user yhuai commented on a diff in the pull request:

    https://github.com/apache/spark/pull/13386#discussion_r65305383
  
    --- Diff: 
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveSessionState.scala ---
    @@ -139,22 +139,6 @@ private[hive] class HiveSessionState(sparkSession: 
SparkSession)
       }
     
       /**
    -   * When true, a table created by a Hive CTAS statement (no USING clause) 
will be
    -   * converted to a data source table, using the data source set by 
spark.sql.sources.default.
    -   * The table in CTAS statement will be converted when it meets any of 
the following conditions:
    -   *   - The CTAS does not specify any of a SerDe (ROW FORMAT SERDE), a 
File Format (STORED AS), or
    -   *     a Storage Handler (STORED BY), and the value of 
hive.default.fileformat in hive-site.xml
    -   *     is either TextFile or SequenceFile.
    -   *   - The CTAS statement specifies TextFile (STORED AS TEXTFILE) as the 
file format and no SerDe
    -   *     is specified (no ROW FORMAT SERDE clause).
    -   *   - The CTAS statement specifies SequenceFile (STORED AS 
SEQUENCEFILE) as the file format
    -   *     and no SerDe is specified (no ROW FORMAT SERDE clause).
    --- End diff --
    
    Yea, the useful part is at 
https://github.com/apache/spark/pull/13386/files#diff-32bb9518401c0948c5ea19377b5069abR313.
 Other parts are not needed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to