GitHub user viirya opened a pull request:

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

    [SPARK-21726][SQL] Check for structural integrity of the plan in Optimzer 
in test mode.

    ## What changes were proposed in this pull request?
    
    We have many optimization rules now in `Optimzer`. Right now we don't have 
any checks in the optimizer to check for the structural integrity of the plan 
(e.g. resolved). When debugging, it is difficult to identify which rules return 
invalid plans.
    
    It would be great if in test mode, we can check whether a plan is still 
resolved after the execution of each rule, so we can catch rules that return 
invalid plans.
    
    ## How was this patch tested?
    
    Added tests.


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

    $ git pull https://github.com/viirya/spark-1 SPARK-21726

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

    https://github.com/apache/spark/pull/18956.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 #18956
    
----
commit 21d86bac80790d0b994df79b5e27a7d2d354e90f
Author: Liang-Chi Hsieh <[email protected]>
Date:   2017-08-16T04:53:49Z

    Check for structural integrity of the plan in Optimzer in test mode.

----


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to