GitHub user yhuai opened a pull request:

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

    [SPARK-14674][SQL] Move HiveContext.hiveconf to HiveSessionState

    ## What changes were proposed in this pull request?
    
    This is just cleanup. This allows us to remove HiveContext later without 
inflating the diff too much. This PR fixes the conflicts of 
https://github.com/apache/spark/pull/12431. It also removes the `def hiveConf` 
from `HiveSqlParser`. So, we will pass the HiveConf associated with a session 
explicitly instead of relying on Hive's `SessionState` to pass `HiveConf`.
    
    
    ## How was this patch tested?
    Existing tests.
    
    Closes #12431

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

    $ git pull https://github.com/yhuai/spark hiveconf

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

    https://github.com/apache/spark/pull/12449.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 #12449
    
----
commit 6e47197431cfae64d760e8b9ce44f8f968fd96cc
Author: Andrew Or <[email protected]>
Date:   2016-04-15T22:12:36Z

    Move hiveconf to HiveSessionState

commit 1436889944a29bd431c275251011cd2f7cd2bb74
Author: Andrew Or <[email protected]>
Date:   2016-04-16T00:18:01Z

    Fix style

commit aeee5bd06b083863776ac17ba8c782890c3e2f6f
Author: Yin Huai <[email protected]>
Date:   2016-04-16T22:49:29Z

    Merge remote-tracking branch 'upstream/master' into hiveconf

commit d817e4113e9959a3555d02b61ee2f28280fbb968
Author: Yin Huai <[email protected]>
Date:   2016-04-16T23:10:19Z

    HiveSqlParser will not rely on Hive's SessionState to get HiveConf.

----


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to