You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by xiaowangyu <gi...@git.apache.org> on 2015/10/14 10:06:04 UTC

[GitHub] spark pull request: SPARK-11100 HiveThriftServer not registering w...

GitHub user xiaowangyu opened a pull request:

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

    SPARK-11100 HiveThriftServer not registering with Zookeeper

    https://issues.apache.org/jira/browse/SPARK-11100

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

    $ git pull https://github.com/jd-ode/spark SPARK-11100

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

    https://github.com/apache/spark/pull/9113.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 #9113
    
----
commit aa4fe7fd60c60ba59e84977668715bec68fe20e4
Author: wangxiaoyu8 <wa...@jd.com>
Date:   2015-10-14T08:03:33Z

    SPARK-11100 HiveThriftServer not registering with Zookeeper

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL] HiveThriftServer not regist...

Posted by andrewor14 <gi...@git.apache.org>.
Github user andrewor14 commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-165224194
  
    ok to test


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #9113: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveThriftSer...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the issue:

    https://github.com/apache/spark/pull/9113
  
    @xiaowangyu the thrift server has gone through a lot of changes lately, and we are not sure whether we'd want to keep the existing thrift server at all because it has a huge amount of technical debt. More so, I'm not sure if the zookeeper approach chosen by Hive is that great, since it is pretty difficult to operate in practice so I'm not sure if this would become the long-term solution. For now, I'd recommend you patching your own version of Spark to get this functionality.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: SPARK-11100 HiveThriftServer not registering w...

Posted by WangTaoTheTonic <gi...@git.apache.org>.
Github user WangTaoTheTonic commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-147977645
  
    with this patch, can we launch multiple thrift server instances and enable client to submit application without sensing particular thrift server addresses instead of zookeeper port?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL]HiveThriftServer HA issue,Hi...

Posted by searchwizard <gi...@git.apache.org>.
Github user searchwizard commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-195047023
  
    Will this be in Spark 1.6.1?
    
    Thanks


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveThriftSe...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the pull request:

    https://github.com/apache/spark/pull/9113
  
    @viper-kun 
    It is use zookeeper to choose the active thrift server when you connect by beeline or jdbc.
    It only happen on the connect time. after connected, if the thrift server is down. the connection should be reconnect by client!


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL] HiveThriftServer not regist...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-165231601
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47849/
    Test PASSed.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #9113: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveThriftSer...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the issue:

    https://github.com/apache/spark/pull/9113
  
    @viper-kun It's random!
    You can check the code from hive:
    https://github.com/apache/hive/blob/release-1.2.1/jdbc/src/java/org/apache/hive/jdbc/ZooKeeperHiveClientHelper.java#L53-L93


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL] HiveThriftServer not regist...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-165225983
  
    **[Test build #47849 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47849/consoleFull)** for PR 9113 at commit [`43769ea`](https://github.com/apache/spark/commit/43769ea659a14919fb432fdd423387a57791c744).


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #9113: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveThriftSer...

Posted by rxin <gi...@git.apache.org>.
Github user rxin commented on the issue:

    https://github.com/apache/spark/pull/9113
  
    We have currently inlined the Hive thrift server into the code base, but the long-term replacement is to be determined. I don't think anybody has thought a lot about it yet.
    
    Do you mind closing this pull request for now? Thanks.



---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL]HiveThriftServer HA issue,Hi...

Posted by chutium <gi...@git.apache.org>.
Github user chutium commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-197969602
  
    +1 to this feature, may need to add this doc http://spark.apache.org/docs/latest/sql-programming-guide.html#running-the-thrift-jdbcodbc-server


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL] HiveThriftServer not regist...

Posted by andrewor14 <gi...@git.apache.org>.
Github user andrewor14 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/9113#discussion_r47681729
  
    --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2.scala ---
    @@ -57,6 +57,11 @@ object HiveThriftServer2 extends Logging {
         val server = new HiveThriftServer2(sqlContext)
         server.init(sqlContext.hiveconf)
         server.start()
    +    if(SparkSQLEnv.hiveContext.hiveconf.getBoolVar(
    --- End diff --
    
    need space after `if`, here and other places


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL] HiveThriftServer not regist...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-165231598
  
    Merged build finished. Test PASSed.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #9113: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveThriftSer...

Posted by viper-kun <gi...@git.apache.org>.
Github user viper-kun commented on the issue:

    https://github.com/apache/spark/pull/9113
  
    @rxin  Is there any design about replacement?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL] HiveThriftServer not regist...

Posted by andrewor14 <gi...@git.apache.org>.
Github user andrewor14 commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-164857542
  
    Jenkins, this is ok to test.
    
    @xiaowangyu can you describe your test setup briefly?
    @liancheng if you have time please review.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: SPARK-11100 HiveThriftServer not registering w...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-147968317
  
    Can one of the admins verify this patch?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #9113: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveThriftSer...

Posted by viper-kun <gi...@git.apache.org>.
Github user viper-kun commented on the issue:

    https://github.com/apache/spark/pull/9113
  
    @xiaowangyu  What I means is how to choose which active thrift server.It chooses random or by thrift server load.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL] HiveThriftServer not regist...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-165116582
  
    Test this issue:
    1.Add property to hive-site.xml
    ```
    hive.server2.support.dynamic.service.discovery=true
    hive.server2.zookeeper.namespace=sparkhiveserver2
    hive.zookeeper.quorum=zk1,zk2,zk3
    ```
    2.Start up sparkthriftserver
    3.In the zookeeper path /sparkhiveserver2 will see:
    ```
    serverUri=x.x.x.x:10000;version=1.2.1.spark;sequence=0000000026
    ```
    4.Connect sparkthriftserver by beeline command:
    ```
    !connect jdbc:hive2://zk1,zk2,zk3/default;serviceDiscoveryMode=zooKeeper;zooKeeperNamespace=sparkhiveserver2
    ```


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #9113: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveThriftSer...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the issue:

    https://github.com/apache/spark/pull/9113
  
    @rxin 
    Thanks! I get it.
    BTW,any component will take the place of the thrift server?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL] HiveThriftServer not regist...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-165231454
  
    **[Test build #47849 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47849/consoleFull)** for PR 9113 at commit [`43769ea`](https://github.com/apache/spark/commit/43769ea659a14919fb432fdd423387a57791c744).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: SPARK-11100 HiveThriftServer not registering w...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-148273396
  
    Yes,My test is OK.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL] HiveThriftServer not regist...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-148587223
  
    @liancheng can you help to review this PR? Thanks very much!


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: SPARK-11100 HiveThriftServer not registering w...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-147989994
  
    Yes,jdbc url like this:
    jdbc:hive2://zk1,zk2,zk3/default;serviceDiscoveryMode=zooKeeper;zooKeeperNamespace=sparkhiveserver2
    see:https://issues.apache.org/jira/browse/HIVE-8376


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #9113: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveThriftSer...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the issue:

    https://github.com/apache/spark/pull/9113
  
    Thanks! I close it.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark issue #9113: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveThriftSer...

Posted by rxin <gi...@git.apache.org>.
Github user rxin commented on the issue:

    https://github.com/apache/spark/pull/9113
  
    @viper-kun no - as I said, "I don't think anybody has thought a lot about it yet."


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL]HiveThriftServer HA issue,Hi...

Posted by DoingDone9 <gi...@git.apache.org>.
Github user DoingDone9 commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-203886680
  
    good job!


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: SPARK-11100 HiveThriftServer not registering w...

Posted by WangTaoTheTonic <gi...@git.apache.org>.
Github user WangTaoTheTonic commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-148255083
  
    Yeah I think it is a nice feature which can bring HA into spark Thrift Server. Have you testet it yet?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request #9113: [SPARK-11100][SQL]HiveThriftServer HA issue,HiveTh...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu closed the pull request at:

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


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL]HiveThriftServer HA issue,Hi...

Posted by xiaowangyu <gi...@git.apache.org>.
Github user xiaowangyu commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-216191887
  
    Any one can merge this patch?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] spark pull request: [SPARK-11100][SQL]HiveThriftServer HA issue,Hi...

Posted by viper-kun <gi...@git.apache.org>.
Github user viper-kun commented on the pull request:

    https://github.com/apache/spark/pull/9113#issuecomment-222619116
  
    @xiaowangyu 
    1.Can you tell me how to work? Thrift server is easy to dump because of too many beelines. If beeline can choose less-load thrift server. 


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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