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

[GitHub] spark pull request: [SPARK-14720][SPARK-13643] Move Hive-specific ...

GitHub user yhuai opened a pull request:

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

    [SPARK-14720][SPARK-13643] Move Hive-specific methods into HiveSessionState and Create a SparkSession class

    ## What changes were proposed in this pull request?
    This PR has two main changes. 
    1. Move Hive-specific methods from HiveContext to HiveSessionState, which help the work of removing HiveContext.
    2. Create a SparkSession Class, which will later be the entry point of Spark SQL users.
    
    
    ## How was this patch tested?
    Existing tests
    
    This PR is based on https://github.com/apache/spark/pull/12485.
    
    Closes #12485


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

    $ git pull https://github.com/yhuai/spark spark-session

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

    https://github.com/apache/spark/pull/12522.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 #12522
    
----
commit f6585f98962403247289c4352954579d14b4e2ed
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-18T17:45:43Z

    [SPARK-14647][SQL] Group SQLContext/HiveContext state into SharedState

commit fe89b8ddc44a32949be6b763e180ef6587d1e310
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-18T19:45:33Z

    Make HiveSessionState take in SQLContext, not HiveContext
    
    This requires changing all the downstream places that take in
    HiveContext and replacing that with (SQLContext, HiveSessionState).

commit 54046d66f68ad06d78bbda73207651c1f29518ab
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-18T20:57:33Z

    Move QueryExecution out of HiveContext

commit 5fc8177e832a01f48da1d558c762a5b522f4496c
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-18T20:58:50Z

    Merge branch 'master' of github.com:apache/spark into spark-session

commit 83b3f7094290613e22713b96a41b937cf9200217
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-18T21:11:54Z

    Minor cleanup

commit b33514c241fb56514cd734caa6741fa8ba1cfb4f
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-18T22:53:17Z

    Implement SparkSession and use it to track state
    
    Now both shared state and session state is tracked in SparkSession
    and we use reflection to instantiate them. After this commit
    SQLContext and HiveContext are just wrappers for SparkSession.

commit 8379143bb8881771771ce2784d447f92df2bfa95
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-18T23:12:05Z

    Merge branch 'master' of github.com:apache/spark into spark-session

commit 6b808aa411ec099b8d856b16ead1d7f9d7c342c0
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-18T23:29:20Z

    Clean up some TODO's and bad signatures

commit 5198955b26c9c6d070a5ffe7292408ed9db2a525
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T00:47:26Z

    Move the bulk of HiveContext into SessionCatalog

commit d58c6afc332622dccde92f588327125c56766446
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T00:51:21Z

    Remove more things from HiveContext

commit edaebe5e5dbf03fe8425b1b4185c1a430b15bece
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T00:54:33Z

    Fix style

commit ce1214d79a90ff212ad871929ed1031350dd8f68
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T00:54:46Z

    Merge branch 'master' of github.com:apache/spark into spark-session

commit 4f3ade994b0e74fff018e17e918f721597c57e9e
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T01:24:56Z

    Minor fixes

commit 60195413d268e5ff7f1c295cc6cbc60794406619
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T01:35:47Z

    Use in-memory catalog by default in tests

commit 75d1115e140b8401343c0b96429ccd346e3a4b17
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T01:47:31Z

    Fix NPE when initializing HiveSessionState

commit 36d6bc891c2d591233e41e470815e0393718af7d
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T01:56:45Z

    Fix the conf

commit 95ffe86daea36b36b22abf6a748bdd99dd4dd4d6
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T17:04:04Z

    Merge branch 'master' of github.com:apache/spark into spark-session

commit a1d45e8f819bead89f5b0b0f68a667d574b250a6
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T18:06:45Z

    Fix REPL in in-memory case
    
    Previously we still tried to load HiveContext even if the user
    explicitly specified an "in-memory" catalog impelmentation. Now
    it will load a SQLContext in this case.

commit 0df39fc9082d2f9f89dc3712e1f49132713f2d38
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T18:20:47Z

    Fix tests: set "in-memory" in more places

commit 0d7309b20f72809659306aec47f29c8585f36dd7
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T18:24:56Z

    Fix style

commit bc352068a40a13712e0f16bbc81d66486e4a3635
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T18:50:27Z

    Fix SQLExecutionSuite
    
    It was failing because we were passing in a subclass of
    SparkContext into SparkSession, and the reflection was using
    the wrong class to get the constructor. This is now fixed with
    ClassTags.

commit 5fcc249f4368963625e294ca3b1ea78b07b96252
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T19:09:43Z

    Fix ParquetHadoopFsRelationSuite
    
    Avoid some unnecessary casts.

commit d9370385ed9d6f88a5aec46b71589cc94bc1f998
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T21:33:42Z

    Fix HiveUDFSuite + refactor TestHive
    
    The problem was that we weren't using the right QueryExecution
    when we called TestHive.sessionState.executePlan. We were using
    HiveQueryExecution instead of the custom one that we created
    in TestHiveContext.
    
    This turned out to be very difficult to fix due to the tight
    coupling of QueryExecution within TestHiveContext. I had to
    refactor this code significantly to extract the nested logic
    one by one.

commit 303f991576f0a082b2d8cb4d98e9a866bb7d07a1
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T21:46:02Z

    Make diff slightly smaller?

commit d27ec500c11e0e7d69e2ac0c7bd9c80901944a63
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T21:50:22Z

    Fix test compile

commit b3d23fa8892ead2508f449063fe8874ab3ebb47d
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T21:50:56Z

    Merge branch 'master' of github.com:apache/spark into spark-session

commit ddc752a134cdea5b06386b0d4a3e7436dc5ba777
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T21:59:47Z

    Fix HiveResolutionSuite

commit 9b8dc3a6efb278b8dceee2f1d491857d79d185db
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T22:09:41Z

    Fix StatisticsSuite

commit e2571372c80a3ef950c16b7a3669affc4154b8ad
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-19T22:49:33Z

    Minor change

commit 8bf123601db1d33455f82d30aa19bdd875bd00d7
Author: Andrew Or <an...@databricks.com>
Date:   2016-04-20T01:06:39Z

    Fix HiveUDFSuite (and many others)
    
    The problem was that we were getting everything from
    executionHive's hiveconf and setting that in metadataHive,
    overriding the value of `hive.metastore.warehouse.dir`,
    which we customize in TestHive. This resulted in a bunch
    of "Table src does not exist" errors from Hive.

----


---
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212371080
  
    **[Test build #2836 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2836/consoleFull)** for PR 12522 at commit [`af42981`](https://github.com/apache/spark/commit/af4298137e7574f29a09e3ac1e731fdb1b343fea).
     * This patch **fails Spark unit 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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212508549
  
    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
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212366110
  
    **[Test build #2835 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2835/consoleFull)** for PR 12522 at commit [`af42981`](https://github.com/apache/spark/commit/af4298137e7574f29a09e3ac1e731fdb1b343fea).
     * This patch **fails PySpark unit 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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212356315
  
    **[Test build #56340 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56340/consoleFull)** for PR 12522 at commit [`af42981`](https://github.com/apache/spark/commit/af4298137e7574f29a09e3ac1e731fdb1b343fea).
     * This patch **fails PySpark unit 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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212578180
  
    **[Test build #56381 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56381/consoleFull)** for PR 12522 at commit [`863976c`](https://github.com/apache/spark/commit/863976cbd42d463be810c88641784717ef8c880f).
     * 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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212508522
  
    **[Test build #56376 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56376/consoleFull)** for PR 12522 at commit [`863976c`](https://github.com/apache/spark/commit/863976cbd42d463be810c88641784717ef8c880f).
     * This patch **fails MiMa 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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212316403
  
    **[Test build #56340 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56340/consoleFull)** for PR 12522 at commit [`af42981`](https://github.com/apache/spark/commit/af4298137e7574f29a09e3ac1e731fdb1b343fea).


---
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212356690
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56340/
    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
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212578756
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56381/
    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 pull request: [SPARK-14720][SPARK-13643] Move Hive-specific ...

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

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


---
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212333963
  
    **[Test build #2836 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2836/consoleFull)** for PR 12522 at commit [`af42981`](https://github.com/apache/spark/commit/af4298137e7574f29a09e3ac1e731fdb1b343fea).


---
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212356684
  
    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
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212580114
  
    Merging this 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 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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212327287
  
    **[Test build #2835 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2835/consoleFull)** for PR 12522 at commit [`af42981`](https://github.com/apache/spark/commit/af4298137e7574f29a09e3ac1e731fdb1b343fea).


---
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212508551
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56376/
    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
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212505416
  
    **[Test build #56376 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56376/consoleFull)** for PR 12522 at commit [`863976c`](https://github.com/apache/spark/commit/863976cbd42d463be810c88641784717ef8c880f).


---
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212578753
  
    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 pull request: [SPARK-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212529275
  
    **[Test build #56381 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56381/consoleFull)** for PR 12522 at commit [`863976c`](https://github.com/apache/spark/commit/863976cbd42d463be810c88641784717ef8c880f).


---
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-14720][SPARK-13643] Move Hive-specific ...

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

    https://github.com/apache/spark/pull/12522#issuecomment-212527381
  
    test 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 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