[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-16 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Thank you, @rxin ! Sure. I'll create a backport PR. --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-16 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15399 Can you create a backport for branch-2.0? --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-16 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15399 Thanks - merging in master. --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67041/ Test PASSed. ---

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #67041 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67041/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #67041 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67041/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-14 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Hi, @rxin or @liancheng . Could you review this PR when you have some time? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-13 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Hi, @rxin . Could you review this again? --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-12 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Hi, @rxin . Since this kind of tests need to change their URI and also assume that the non-default database exists prior, I made a new testsuite for this. Could you review this again?

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66841/ Test PASSed. ---

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66841 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66841/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66841 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66841/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-10 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Or, should I create a new testsuite for this kind of testcases? --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-10 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Thank you for your advice, @rxin !! Which suite do you mean? The suites inside `HiveThriftServer2Suites.scala` look not proper to me for this kind of cases. --- If your project is set up

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-10 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/15399 Can we add a test case using the jdbc 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-10 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Hi, @liancheng . Could you review this PR about 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-10 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Thank you for commenting, @gatorsmile . So far, I asked @rxin and you. I'll find another committer more closer this part. --- If your project is set up for it, you can reply to this email

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-09 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/15399 First, I am not familiar with the code in this component. Thus, I am not the right person to review it. Second, when I going over the pending JIRA list, I found many bugs that are

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66617/ Test PASSed. ---

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-09 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66617 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66617/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-09 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66617 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66617/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-08 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Hi, @gatorsmile . Could you review this PR when you have sometime? --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66560/ Test PASSed. ---

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66560 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66560/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66560 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66560/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Merged build finished. Test FAILed. --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66557 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66557/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66557/ Test FAILed. ---

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66557 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66557/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Retest this please. --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Merged build finished. Test FAILed. --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66555 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66555/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66555/ Test FAILed. ---

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66555 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66555/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Oops. Sorry. I'll fix soon. --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15399 **[Test build #66551 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66551/consoleFull)** for PR 15399 at commit

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66551/ Test FAILed. ---

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15399 Merged build finished. Test FAILed. --- 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

[GitHub] spark issue #15399: [SPARK-17819][SQL] Support default database in connectio...

2016-10-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/15399 Hi, @rxin . Sorry, but could you give me some advice about a proper testsuite for this kind of issue? --- If your project is set up for it, you can reply to this email and have your