GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/14821
[SPARK-17250] [SQL] Remove HiveClient and setCurrentDatabase from
HiveSessionCatalog
### What changes were proposed in this pull request?
This is the first step to remove `HiveClient` from `HiveSessionState`. In
the metastore interaction, we always set fully qualified names when
accessing/operating a table. That means, we always specify the database. Thus,
it is not necessary to use `HiveClient` to change the active database in Hive
metastore.
In `HiveSessionCatalog `, `setCurrentDatabase` is the only function that
uses `HiveClient`. Thus, we can remove it after removing `setCurrentDatabase`
### How was this patch tested?
The existing test cases.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gatorsmile/spark setCurrentDB
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/14821.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 #14821
----
commit 1c9a1e3c608be72ca7c4203ecc0e80f15080eb80
Author: gatorsmile <gatorsm...@gmail.com>
Date: 2016-08-26T00:51:57Z
fix
----
---
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
