GitHub user vipul1409 reopened a pull request:

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

    [ZEPPELIN-2519] Disable watchers not in viewport

    ### What is this PR for?
    Currently all the watchers are enabled by default. I came across this 
github project https://github.com/wix/angular-viewport-watch to disable 
watchers not in viewport. This reduces number of watchers in notebooks with 
large number of paragraphs.
    
    
    ### What type of PR is it?
    [Improvement]
    
    ### Todos
    
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-2519
    
    ### How should this be tested?
    Outline the steps to test the PR here.
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update?
    Not sure of this part. This is under MIT license. [Project 
link](https://github.com/wix/angular-viewport-watch)
    
    * 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/vipul1409/zeppelin ZEPPELIN-2519

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

    https://github.com/apache/zeppelin/pull/2326.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 #2326
    
----
commit 329e33a27b28b494f739c0d569bb4ea752e9044a
Author: vipulm <vip...@qubole.com>
Date:   2017-05-09T15:29:21Z

    Disable watchers not in viewport

commit 71425782202f57a41b41f743f6e2c0c3b1b526e8
Author: vipulm <vip...@qubole.com>
Date:   2017-05-09T17:16:17Z

    Trigger notification

commit d698d066d186b9c3c2b0b469632c4cadf5360320
Author: vipulm <vip...@qubole.com>
Date:   2017-05-10T06:18:34Z

    Add license file

commit 99b19a5ee018d47bc7d076f6688ef102918debc6
Author: vipulm <vip...@qubole.com>
Date:   2017-05-11T05:51:00Z

    Move pacakg to npm

commit 175dd1768c554e97d646ed08ccdc52eba8395b06
Author: vipulm <vip...@qubole.com>
Date:   2017-05-16T05:44:10Z

    Merge master

commit b982db4915176b718f561838c3a7163831367408
Author: vipulm <vip...@qubole.com>
Date:   2017-05-19T05:44:48Z

    Trigger build

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to