[GitHub] zeppelin pull request #3204: [ZEPPELIN-3694] Dynamically change scheduler th...

[egorklimov]

GitHub user egorklimov opened a pull request:

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

     [ZEPPELIN-3694] Dynamically change scheduler thread pool size using REST 
API 

    ### What is this PR for?
    This PR is reopening of https://github.com/apache/zeppelin/pull/3121 
    It would be nice if we could change thread pool size for scheduler 
dynamically.
    
    ### Using this feature:
    1. Create file `quartz.properties` from default file located in 
`org.quartz` and add them to `conf` directory
    2. Notice, that **org.quartz.threadPool.threadPriority** and 
**org.quartz.threadPool.threadsInheritContextClassLoaderOfInitializingThread** 
are not implemented in **ExecutorServiceThreadPool** so these options should be 
deleted from the file
    3. In `quartz.properties` set `org.quartz.threadPool.class` to 
`org.apache.zeppelin.scheduler.dynamic_pool.impl.ExecutorServiceThreadPool`
    
    Otherwise you will get message "Thread pool size is constant."
    
    ### What type of PR is it?
    Improvement
    
    ### What is the Jira issue?
    * issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN-3694
    
    ### How should this be tested?
    * Tests added
    * Tested manually
    * [CI 
pass](https://travis-ci.org/TinkoffCreditSystems/zeppelin/builds/441237953)
    
    ### Screenshots
    Firstly, when thread pool size equals `1` notes wait for each other, but 
when scheduler thread pool size changed to `2` they start to work in parallel.
    
    ![cron 
mp4](https://user-images.githubusercontent.com/6136993/43912629-77707ad4-9c0b-11e8-9d6c-62cbc67568a8.gif)
    
    
    
    ### Questions:
    * Does the licenses files need update?  No
    I added sources of 
[quartz-dynamic-pool](https://github.com/epiresdasilva/quartz-dynamic-pool) 
developed by @epiresdasilva, because this repo isn't included in 
http://mvnrepository.com
    * Is there breaking changes for older versions? No
    * Does this needs documentation? Yes, REST API docs updated


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

    $ git pull https://github.com/TinkoffCreditSystems/zeppelin DW-17795

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

    https://github.com/apache/zeppelin/pull/3204.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 #3204
    
----
commit b751ea79df1f4766d755abcf660a614e6c3ff935
Author: egorklimov <klim.electronicmail@...>
Date:   2018-08-03T10:24:24Z

    Quartz-dynamic pool added

----


---