You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by hammertank <gi...@git.apache.org> on 2017/04/24 02:24:16 UTC

[GitHub] zeppelin pull request #2277: [HOTFIX] Fix CI build failure on branch-0.6

GitHub user hammertank opened a pull request:

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

    [HOTFIX] Fix CI build failure on branch-0.6

    ### What is this PR for?
    Fix CI build failure on branch-0.6. 
    
    ### What type of PR is it?
    Hot Fix
    
    ### Todos
    - Modified [.travis.yml](https://github.com/apache/zeppelin/blob/branch-0.6/.travis.yml) according to #1774 
    - Merged PR #1920 #1911
    - Update spark version and hadoop version in [.travis.yml](https://github.com/apache/zeppelin/blob/branch-0.6/.travis.yml) since older spark version below 1.6.2 can not be downloaded from remote server any more.
    
    ### What is the Jira issue?
    No related Jira issue
    
    ### How should this be tested?
    See if CI becomes green
    
    ### 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/hammertank/zeppelin branch-0.6

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

    https://github.com/apache/zeppelin/pull/2277.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 #2277
    
----
commit 4ae3adb24424d8d79969a43bbf6489f4c662afe3
Author: Lee moon soo <mo...@apache.org>
Date:   2017-01-20T21:26:25Z

    [ZEPPELIN-1455] Fix flaky test: AbstractAngularElemTest
    
    ### What is this PR for?
    This PR fix flaky test [ZEPPELIN-1455](https://issues.apache.org/jira/browse/ZEPPELIN-1455).
    
    According to http://doc.scalatest.org/1.8/org/scalatest/concurrent/Eventually.html, default timeout of eventually is 150millisecond. Set enough timeout for the test.
    
    ### What type of PR is it?
    Hot Fix
    
    ### Todos
    * [x] - increase timeout
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-1455
    
    ### Questions:
    * Does the licenses files need update? no
    * Is there breaking changes for older versions? no
    * Does this needs documentation? no
    
    Author: Lee moon soo <mo...@apache.org>
    
    Closes #1920 from Leemoonsoo/ZEPPELIN-1455 and squashes the following commits:
    
    13a993d [Lee moon soo] Increase tolerance of eventually
    
    (cherry picked from commit 99c21c4)

commit f1c8041f4a9b7122d64076c5b1b014a08128629a
Author: Igor Drozdov <ig...@epam.com>
Date:   2017-01-16T09:19:35Z

    [ZEPPELIN-1972] Preserve context classloader
    
    ### What is this PR for?
    `IMain.interpret` changes current thread's context classloader. It may cause different issues and definitely is the reason of [ZEPPELIN-1738](https://issues.apache.org/jira/browse/ZEPPELIN-1738) test failures.
    
    It's a known scala bug. See https://issues.scala-lang.org/browse/SI-9587
    
    As a workaround we need to save and restore context classloader manually
    
    ### What type of PR is it?
    Bug Fix
    
    ### What is the Jira issue?
    [ZEPPELIN-1972](https://issues.apache.org/jira/browse/ZEPPELIN-1972)
    
    ### How should this be tested?
    Run ignite interpreter test and ignite sql interpreter test in the same thread
    ```
    mvn test -pl ignite -am -Pscala-2.11 -Dtest=org.apache.zeppelin.ignite.IgniteInterpreterTest#testInterpret,org.apache.zeppelin.ignite.IgniteSqlInterpreterTest#testSql -DfailIfNoTests=false
    ```
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    
    Author: Igor Drozdov <ig...@epam.com>
    
    Closes #1911 from DrIgor/ZEPPELIN-1972 and squashes the following commits:
    
    35c5abd [Igor Drozdov] Preserve context classloader
    
    (cherry picked from commit 859d175)

commit 380432c4cc1cfa0f2b49ddbd06c7c463fe1d633a
Author: z0621 <zh...@dtdream.com>
Date:   2017-04-24T01:42:21Z

    [HOTFIX] Fix CI build failure on branch-0.6

----


---
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 #2277: [HOTFIX] Fix CI build failure on branch-0.6

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

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


---
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 #2277: [HOTFIX] Fix CI build failure on branch-0.6

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

    https://github.com/apache/zeppelin/pull/2277
  
    @Leemoonsoo I closed the old PR #2262 and created this one with commits' original author info. Is this OK?


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