GitHub user liufengdb opened a pull request:

    [SPARK-23275][SQL] fix the thread leaking in hive/tests

    ## What changes were proposed in this pull request?
    The two lines actually can trigger the hive metastore bug:
    The two configs are not in the default `ObjectStore` properties, so any run 
hive commands after these two lines will set the `propsChanged` flag in the 
`ObjectStore.setConf` and then cause thread leaks.
    I don't think two lines are very useful and can be removed completed.
    ## How was this patch tested?
    (Please explain how this patch was tested. E.g. unit tests, integration 
tests, manual tests)
    (If this patch involves UI changes, please attach a screenshot; otherwise, 
remove this)
    Please review before opening a 
pull request.

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

    $ git pull fix-omm

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

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #20562
commit 4e10f34ebb17940a1f0fd54ce26cb16d06320770
Author: Feng Liu <fengliu@...>
Date:   2018-02-09T21:15:36Z

    init: remove the two lines



To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to