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

[GitHub] spark pull request: [SQL] SPARK-6981: Factor out SparkPlanner and ...

GitHub user evacchi opened a pull request:

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

    [SQL] SPARK-6981: Factor out SparkPlanner and QueryExecution from SQLContext

    Dependent types add additional, unnecessary complexity to third-parties who may want to extend SQLContext with new rewriting strategies; moreover, HiveContext benefits from this simplifying change as well.

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

    $ git pull https://github.com/evacchi/spark sqlctx-refactor

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

    https://github.com/apache/spark/pull/5556.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 #5556
    
----
commit 114f5a1ac0a70ae61dab35efc7e657050daaa631
Author: Edoardo Vacchi <ed...@unicredit.eu>
Date:   2015-04-17T13:18:18Z

    Refactor out SparkPlanner from SQLContext
    
    Signed-off-by: Edoardo Vacchi <Ed...@unicredit.eu>

commit 131e8e9c5595fa84536ab7f6a5b5868e0fdd464d
Author: Edoardo Vacchi <ed...@unicredit.eu>
Date:   2015-04-17T13:24:13Z

    Cleanup HiveContext, following SparkContext refactoring
    
    Signed-off-by: Edoardo Vacchi <Ed...@unicredit.eu>

commit 61478b51985fb5fcd32d4c8c277706831cb4853f
Author: Edoardo Vacchi <ed...@unicredit.eu>
Date:   2015-04-17T13:30:42Z

    Refactor out QueryExecution from SQLContext
    
    Signed-off-by: Edoardo Vacchi <Ed...@unicredit.eu>

commit ed6a563b8df49de694312551213eaafe56cea552
Author: Edoardo Vacchi <ed...@unicredit.eu>
Date:   2015-04-17T13:37:03Z

    Factor out HiveQueryExecution from HiveContext
    
    Signed-off-by: Edoardo Vacchi <Ed...@unicredit.eu>

commit 59293c4f5784208e98e471b002d89777b1245d74
Author: Edoardo Vacchi <ed...@unicredit.eu>
Date:   2015-04-17T13:46:20Z

    Revert erroneous test rename
    
    Signed-off-by: Edoardo Vacchi <Ed...@unicredit.eu>

----


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94592922
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30612/
    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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94702538
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30655/
    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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94737235
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30660/
    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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-95296434
  
    Sorry, it is possible that you hit a flakey test.  If you fix the merge conflicts we can try 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 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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94702010
  
      [Test build #30655 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30655/consoleFull) for   PR 5556 at commit [`7cc8fd5`](https://github.com/apache/spark/commit/7cc8fd5c25296fe11ae8c1b79d21303a609cd58e).


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-93990147
  
    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 pull request: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-95515162
  
      [Test build #30825 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30825/consoleFull) for   PR 5556 at commit [`77a5df4`](https://github.com/apache/spark/commit/77a5df4e16fad6a05e0b96966b04d0fbf16d7a4c).
     * This patch **passes all tests**.
     * This patch merges cleanly.
     * This patch adds the following public classes _(experimental)_:
      * `trait QueryPlanner[PhysicalPlan <: TreeNode[PhysicalPlan]] `
      * `abstract class NumericType extends NativeType `
      * `protected[sql] class QueryExecution(val sqlContext: SQLContext, val logical: LogicalPlan) `
      * `protected[sql] class SparkPlanner(val sqlContext: SQLContext) extends SparkStrategies `
      * `protected[sql] class HiveQueryExecution(hiveContext: HiveContext, logicalPlan: LogicalPlan)`
    
     * This patch does not change any dependencies.


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-96766989
  
    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 pull request: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-95485095
  
      [Test build #30825 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30825/consoleFull) for   PR 5556 at commit [`77a5df4`](https://github.com/apache/spark/commit/77a5df4e16fad6a05e0b96966b04d0fbf16d7a4c).


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94706966
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30656/
    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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94706961
  
      [Test build #30656 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30656/consoleFull) for   PR 5556 at commit [`5a0a9f8`](https://github.com/apache/spark/commit/5a0a9f882160814d83c697ff90c08c41e7c48897).
     * This patch **fails to build**.
     * This patch merges cleanly.
     * This patch adds the following public classes _(experimental)_:
      * `trait QueryPlanner[PhysicalPlan <: TreeNode[PhysicalPlan]] `
      * `protected[sql] class QueryExecution(val sqlContext: SQLContext, val logical: LogicalPlan) `
      * `protected[sql] class SparkPlanner(val sqlContext: SQLContext) extends SparkStrategies `
      * `protected[sql] class HiveQueryExecution(hiveContext: HiveContext, logicalPlan: LogicalPlan)`
    
     * This patch does not change any dependencies.


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94592909
  
      [Test build #30612 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30612/consoleFull) for   PR 5556 at commit [`0b29d80`](https://github.com/apache/spark/commit/0b29d809de41f18259e23b975ef34197ad1112f7).


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94592384
  
    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 pull request: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94592920
  
      [Test build #30612 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30612/consoleFull) for   PR 5556 at commit [`0b29d80`](https://github.com/apache/spark/commit/0b29d809de41f18259e23b975ef34197ad1112f7).
     * This patch **fails RAT tests**.
     * This patch merges cleanly.
     * This patch adds the following public classes _(experimental)_:
      * `trait QueryPlanner[PhysicalPlan <: TreeNode[PhysicalPlan]] `
      * `protected[sql] class QueryExecution(val sqlContext: SQLContext, val logical: LogicalPlan) `
      * `protected[sql] class SparkPlanner(val sqlContext: SQLContext) extends SparkStrategies `
      * `protected[sql] class HiveQueryExecution(hiveContext: HiveContext, logicalPlan: LogicalPlan)`
    
     * This patch does not change any dependencies.


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-95181457
  
    I can confirm that the pyspark/sql/dataframe.py test case passes successfully on my branch.


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94737211
  
      [Test build #30660 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30660/consoleFull) for   PR 5556 at commit [`95a26c3`](https://github.com/apache/spark/commit/95a26c35cc77b49f12a2a29090896b917470d661).
     * This patch **fails PySpark unit tests**.
     * This patch merges cleanly.
     * This patch adds the following public classes _(experimental)_:
      * `trait QueryPlanner[PhysicalPlan <: TreeNode[PhysicalPlan]] `
      * `protected[sql] class QueryExecution(val sqlContext: SQLContext, val logical: LogicalPlan) `
      * `protected[sql] class SparkPlanner(val sqlContext: SQLContext) extends SparkStrategies `
      * `protected[sql] class HiveQueryExecution(hiveContext: HiveContext, logicalPlan: LogicalPlan)`
    
     * This patch does not change any dependencies.


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94738010
  
    I may be mistaken here but my code does not change any of PySpark, and the stack trace seems to be pointing at a config error?
    
    > akka.ConfigurationException: Logger specified in config can't be loaded [akka.event.slf4j.Slf4jLogger] due to [akka.event.Logging$LoggerInitializationException: Logger log1-Slf4jLogger did not respond with LoggerInitialized, sent instead [TIMEOUT]]



---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-95515194
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30825/
    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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94713490
  
      [Test build #30660 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30660/consoleFull) for   PR 5556 at commit [`95a26c3`](https://github.com/apache/spark/commit/95a26c35cc77b49f12a2a29090896b917470d661).


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94702534
  
      [Test build #30655 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30655/consoleFull) for   PR 5556 at commit [`7cc8fd5`](https://github.com/apache/spark/commit/7cc8fd5c25296fe11ae8c1b79d21303a609cd58e).
     * This patch **fails Scala style tests**.
     * This patch merges cleanly.
     * This patch adds the following public classes _(experimental)_:
      * `trait QueryPlanner[PhysicalPlan <: TreeNode[PhysicalPlan]] `
      * `protected[sql] class QueryExecution(val sqlContext: SQLContext, val logical: LogicalPlan) `
      * `protected[sql] class SparkPlanner(val sqlContext: SQLContext) extends SparkStrategies `
      * `protected[sql] class HiveQueryExecution(hiveContext: HiveContext, logicalPlan: LogicalPlan)`
    
     * This patch does not change any dependencies.


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-95485118
  
    I tried to rebase. Hope it's not a problem.


---
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: [SQL] SPARK-6981: Factor out SparkPlanner and ...

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

    https://github.com/apache/spark/pull/5556#issuecomment-94704801
  
      [Test build #30656 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30656/consoleFull) for   PR 5556 at commit [`5a0a9f8`](https://github.com/apache/spark/commit/5a0a9f882160814d83c697ff90c08c41e7c48897).


---
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