GitHub user yaooqinn opened a pull request:

    https://github.com/apache/spark/pull/19712

    [SPARK-22487][SQL][Hive]Remove the unused HIVE_EXECUTION_VERSION property

    ## What changes were proposed in this pull request?
    
    Actually there is no hive client for executions in spark now and there are 
no usages of HIVE_EXECUTION_VERSION found in whole spark project. 
HIVE_EXECUTION_VERSION is set by `spark.sql.hive.version`, which is still set 
internally in some places or by users, this may confuse developers and users 
with HIVE_METASTORE_VERSION(spark.sql.hive.metastore.version).
    
    It might better to be removed.
    
    ## How was this patch tested?
    
    modify some existing ut 
    
    cc @cloud-fan @gatorsmile 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/yaooqinn/spark SPARK-22487

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/19712.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 #19712
    
----
commit 607192603b88f6ed4543587489188f20b9b236e0
Author: Kent Yao <yaooq...@hotmail.com>
Date:   2017-11-10T03:06:32Z

    rm unused hive_execution_version

----


---

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

Reply via email to