You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by jongyoul <gi...@git.apache.org> on 2016/07/18 09:12:32 UTC

[GitHub] zeppelin pull request #1201: [MINOR] Enable pyspark test in local mode

GitHub user jongyoul opened a pull request:

    https://github.com/apache/zeppelin/pull/1201

    [MINOR] Enable pyspark test in local mode

    ### What is this PR for?
    Enabling test for pyspark in local mode
    
    ### What type of PR is it?
    [Improvement]
    
    ### Todos
    * [x] - Add spark configuration and
    * [x] - Edit a logic for finding spark home 
    
    ### What is the Jira issue?
    N/A
    
    ### How should this be tested?
    1. Download spark under {ZEPPELIN_HOME}
    1. run `testing/startSparkCluster.sh`
    1. `mvn clean package -Pspark-1.6 -Ppyspark -Pyarn -DskipTests`
    1. Run `ZeppelinSparkClusterTest`
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    


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

    $ git pull https://github.com/jongyoul/zeppelin minor/enable-pyspark-test-in-local-mode

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

    https://github.com/apache/zeppelin/pull/1201.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 #1201
    
----
commit 67ccc2df0397267bb4d3e3140c24cd1c1efa78da
Author: Jongyoul Lee <jo...@gmail.com>
Date:   2016-07-18T09:09:42Z

    Added spark configuration while not using CI

----


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

[GitHub] zeppelin pull request #1201: [MINOR] Enable pyspark test in local mode

Posted by jongyoul <gi...@git.apache.org>.
GitHub user jongyoul reopened a pull request:

    https://github.com/apache/zeppelin/pull/1201

    [MINOR] Enable pyspark test in local mode

    ### What is this PR for?
    Enabling test for pyspark in local mode
    
    ### What type of PR is it?
    [Improvement]
    
    ### Todos
    * [x] - Add spark configuration and
    * [x] - Edit a logic for finding spark home 
    
    ### What is the Jira issue?
    N/A
    
    ### How should this be tested?
    1. Download spark under {ZEPPELIN_HOME}
    1. run `testing/startSparkCluster.sh`
    1. `mvn clean package -Pspark-1.6 -Ppyspark -Pyarn -DskipTests`
    1. Run `ZeppelinSparkClusterTest`
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    


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

    $ git pull https://github.com/jongyoul/zeppelin minor/enable-pyspark-test-in-local-mode

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

    https://github.com/apache/zeppelin/pull/1201.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 #1201
    
----
commit 67ccc2df0397267bb4d3e3140c24cd1c1efa78da
Author: Jongyoul Lee <jo...@gmail.com>
Date:   2016-07-18T09:09:42Z

    Added spark configuration while not using CI

----


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

[GitHub] zeppelin issue #1201: [MINOR] Enable pyspark test in local mode

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

    https://github.com/apache/zeppelin/pull/1201
  
    LGTM. 
    
    CI failure is flaky one. i've seen it couple of times.


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

[GitHub] zeppelin issue #1201: [MINOR] Enable pyspark test in local mode

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

    https://github.com/apache/zeppelin/pull/1201
  
    I've seen it for the first time. Do you have any hint to deal with?


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

[GitHub] zeppelin issue #1201: [MINOR] Enable pyspark test in local mode

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

    https://github.com/apache/zeppelin/pull/1201
  
    Looks great to me!
    
    CI fails on
    ```
    [INFO] Zeppelin: Display system apis ...................... FAILURE [ 24.426 s]
    
    [31m- should provide onclick method *** FAILED ***
      The code passed to eventually never returned normally. Attempted 1 times over 361.487655 milliseconds. Last failure message: 0 was not equal to 1. (AbstractAngularElemTest.scala:72)
    AngularElem
    
    [36mTests: succeeded 17, failed 1, canceled 0, ignored 0, pending 0
    *** 1 TEST FAILED ***
    ```
    
    Have you seen this before is it completely random flaky 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.
---

[GitHub] zeppelin pull request #1201: [MINOR] Enable pyspark test in local mode

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

    https://github.com/apache/zeppelin/pull/1201


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

[GitHub] zeppelin pull request #1201: [MINOR] Enable pyspark test in local mode

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

    https://github.com/apache/zeppelin/pull/1201


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

[GitHub] zeppelin issue #1201: [MINOR] Enable pyspark test in local mode

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

    https://github.com/apache/zeppelin/pull/1201
  
    Thanks. I'll merge 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.
---