[GitHub] carbondata pull request #1822: [CARBONDATA-2043] Configurable wait time for ...

[qiuchenjian-2]

GitHub user mohammadshahidkhan opened a pull request:

    https://github.com/apache/carbondata/pull/1822

    [CARBONDATA-2043] Configurable wait time for requesting executors and…

    **Problem**
    In case of spark.dynamicAllocation.enabled to true, during the block distribution based on the needs of required number of executors carbon request the executors from sparks and waits for specified time
    for required executor's to come up before starting the block distribution.
    Carbon waits for the specified time even if not possible to get the required number of executors.
    **Solution**
    Configurable wait time for requesting executors and minimum registered executors ratio to continue the block distribution
    - carbon.dynamicAllocation.schedulerTimeout : to configure wait time. defalt 5sec, Min 5 sec and max 15 sec.
    - carbon.scheduler.minRegisteredResourcesRatio :    min 0.1, max 1.0 and default to 0.8 to configure minimum registered executors ratio.
   
    Be sure to do all of the following checklist to help us incorporate
    your contribution quickly and easily:
   
     - [X] Any interfaces changed?
        No
   
     - [X] Any backward compatibility impacted?
     
     - [X] Document update required?
        Yes
     - [X] Testing done
            Please provide details on
            - Whether new unit test cases have been added or why no new tests are required?
            - How it is tested? Please attach test report.
            - Is it a performance related change? Please attach the performance test report.
            - Any additional information to help reviewers in testing this change.
           Added test case for the property configuration validation.
     - [X] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
    NA


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

    $ git pull https://github.com/mohammadshahidkhan/incubator-carbondata scheduler_wait_time_for_executors

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

    https://github.com/apache/carbondata/pull/1822.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 #1822
   
----
commit f708ab9790debad1ff6937ad90d7e955fb143544
Author: mohammadshahidkhan <mohdshahidkhan1987@...>
Date:   2017-12-22T12:00:31Z

    [CARBONDATA-2043] Configurable wait time for requesting executors and minimum registered executors ratio to continue the block distribution
    - carbon.dynamicAllocation.schedulerTimeout : to configure wait time. defalt 5sec, Min 5 sec and max 15 sec.
    - carbon.scheduler.minRegisteredResourcesRatio :     min 0.1, max 1.0 and default to 0.8 to configure minimum registered executors ratio.

----


---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2889/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1656/



---

[qiuchenjian-2]

Github user mohammadshahidkhan commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    retest this please


---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1698/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2931/



---

[qiuchenjian-2]

Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/2953/



---

[qiuchenjian-2]

Github user manishgupta88 commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    retest sdv please


---

[qiuchenjian-2]

Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/2988/



---

[qiuchenjian-2]

Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/3037/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1815/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/3046/



---

[qiuchenjian-2]

Github user kushalsaha commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    retest SDV this please


---

[qiuchenjian-2]

Github user kushalsaha commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    retest SDV please


---

[qiuchenjian-2]

Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/3048/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/3283/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/2053/



---

[qiuchenjian-2]

Github user mohammadshahidkhan commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    retest SDV please


---

[qiuchenjian-2]

Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/3233/



---

[qiuchenjian-2]

Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1822
 
    SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/3240/



---