You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Erik Bergenholtz <eb...@hortonworks.com> on 2015/05/28 05:11:38 UTC

Review Request 34751: CapScheduler Auto-Create and Cluster association

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34751/
-----------------------------------------------------------

Review request for Ambari and Robert Levas.


Bugs: AMBARI-11436
    https://issues.apache.org/jira/browse/AMBARI-11436


Repository: ambari


Description
-------

Capacity Scheduler needs to support cluster association (configure based on cluster) and auto instance creation.


Diffs
-----

  contrib/views/capacity-scheduler/pom.xml 2712953 
  contrib/views/capacity-scheduler/readme.md e772508 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/ConfigurationService.java 8e1b6a6 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/Proxy.java c026b41 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/RequestBuilder.java 88e19bb 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/ResponseTranslator.java 8e3b4a6 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/components/dropdownButtons.js cc15563 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js 9828399 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js b26697f 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 4dfb5e5 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less 82fbc07 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueContainer.hbs cdc9044 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/queues.hbs b412d7b 
  contrib/views/capacity-scheduler/src/main/resources/ui/test/integration/serializers_test.js PRE-CREATION 
  contrib/views/capacity-scheduler/src/main/resources/view.xml 2974f86 
  contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/AmbariApi.java 88e5f48 
  contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/URLStreamProviderBasicAuth.java 87a4acb 

Diff: https://reviews.apache.org/r/34751/diff/


Testing
-------

Local Unit Tests, deployment locally of built view to ensure that cluster association & auto-create works upon adding YARN service.


Thanks,

Erik Bergenholtz


Re: Review Request 34751: CapScheduler Auto-Create and Cluster association

Posted by Robert Levas <rl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34751/#review85560
-----------------------------------------------------------

Ship it!


Ship It!

- Robert Levas


On May 28, 2015, 11:33 a.m., Erik Bergenholtz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34751/
> -----------------------------------------------------------
> 
> (Updated May 28, 2015, 11:33 a.m.)
> 
> 
> Review request for Ambari and Robert Levas.
> 
> 
> Bugs: AMBARI-11436
>     https://issues.apache.org/jira/browse/AMBARI-11436
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Capacity Scheduler needs to support cluster association (configure based on cluster) and auto instance creation.
> 
> 
> Diffs
> -----
> 
>   contrib/views/capacity-scheduler/pom.xml 2712953 
>   contrib/views/capacity-scheduler/readme.md e772508 
>   contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/ConfigurationService.java 8e1b6a6 
>   contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/Proxy.java c026b41 
>   contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/RequestBuilder.java 88e19bb 
>   contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/ResponseTranslator.java 8e3b4a6 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/components/dropdownButtons.js cc15563 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js 9828399 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js b26697f 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 4dfb5e5 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less 82fbc07 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueContainer.hbs cdc9044 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/queues.hbs b412d7b 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/refuse.hbs 8671d7d 
>   contrib/views/capacity-scheduler/src/main/resources/ui/config.coffee 0c86396 
>   contrib/views/capacity-scheduler/src/main/resources/ui/test/integration/serializers_test.js PRE-CREATION 
>   contrib/views/capacity-scheduler/src/main/resources/view.xml 2974f86 
>   contrib/views/pom.xml 9fa698b 
>   contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/AmbariApi.java 88e5f48 
>   contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/URLStreamProviderBasicAuth.java 87a4acb 
> 
> Diff: https://reviews.apache.org/r/34751/diff/
> 
> 
> Testing
> -------
> 
> Local Unit Tests, deployment locally of built view to ensure that cluster association & auto-create works upon adding YARN service.
> 
> 
> Thanks,
> 
> Erik Bergenholtz
> 
>


Re: Review Request 34751: CapScheduler Auto-Create and Cluster association

Posted by Erik Bergenholtz <eb...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34751/
-----------------------------------------------------------

(Updated May 28, 2015, 3:33 p.m.)


Review request for Ambari and Robert Levas.


Changes
-------

New Patch addresses build issue where the dependent utils jar is not included in the derived jar.


Bugs: AMBARI-11436
    https://issues.apache.org/jira/browse/AMBARI-11436


Repository: ambari


Description
-------

Capacity Scheduler needs to support cluster association (configure based on cluster) and auto instance creation.


Diffs (updated)
-----

  contrib/views/capacity-scheduler/pom.xml 2712953 
  contrib/views/capacity-scheduler/readme.md e772508 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/ConfigurationService.java 8e1b6a6 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/Proxy.java c026b41 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/RequestBuilder.java 88e19bb 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/ResponseTranslator.java 8e3b4a6 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/components/dropdownButtons.js cc15563 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js 9828399 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js b26697f 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 4dfb5e5 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less 82fbc07 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueContainer.hbs cdc9044 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/queues.hbs b412d7b 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/refuse.hbs 8671d7d 
  contrib/views/capacity-scheduler/src/main/resources/ui/config.coffee 0c86396 
  contrib/views/capacity-scheduler/src/main/resources/ui/test/integration/serializers_test.js PRE-CREATION 
  contrib/views/capacity-scheduler/src/main/resources/view.xml 2974f86 
  contrib/views/pom.xml 9fa698b 
  contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/AmbariApi.java 88e5f48 
  contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/URLStreamProviderBasicAuth.java 87a4acb 

Diff: https://reviews.apache.org/r/34751/diff/


Testing
-------

Local Unit Tests, deployment locally of built view to ensure that cluster association & auto-create works upon adding YARN service.


Thanks,

Erik Bergenholtz


Re: Review Request 34751: CapScheduler Auto-Create and Cluster association

Posted by Robert Levas <rl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34751/#review85548
-----------------------------------------------------------

Ship it!


Ship It!

- Robert Levas


On May 28, 2015, 6:48 a.m., Erik Bergenholtz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34751/
> -----------------------------------------------------------
> 
> (Updated May 28, 2015, 6:48 a.m.)
> 
> 
> Review request for Ambari and Robert Levas.
> 
> 
> Bugs: AMBARI-11436
>     https://issues.apache.org/jira/browse/AMBARI-11436
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Capacity Scheduler needs to support cluster association (configure based on cluster) and auto instance creation.
> 
> 
> Diffs
> -----
> 
>   contrib/views/capacity-scheduler/pom.xml 2712953 
>   contrib/views/capacity-scheduler/readme.md e772508 
>   contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/ConfigurationService.java 8e1b6a6 
>   contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/Proxy.java c026b41 
>   contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/RequestBuilder.java 88e19bb 
>   contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/ResponseTranslator.java 8e3b4a6 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/components/dropdownButtons.js cc15563 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js 9828399 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js b26697f 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 4dfb5e5 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less 82fbc07 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueContainer.hbs cdc9044 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/queues.hbs b412d7b 
>   contrib/views/capacity-scheduler/src/main/resources/ui/test/integration/serializers_test.js PRE-CREATION 
>   contrib/views/capacity-scheduler/src/main/resources/view.xml 2974f86 
>   contrib/views/pom.xml 9fa698b 
>   contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/AmbariApi.java 88e5f48 
>   contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/URLStreamProviderBasicAuth.java 87a4acb 
> 
> Diff: https://reviews.apache.org/r/34751/diff/
> 
> 
> Testing
> -------
> 
> Local Unit Tests, deployment locally of built view to ensure that cluster association & auto-create works upon adding YARN service.
> 
> 
> Thanks,
> 
> Erik Bergenholtz
> 
>


Re: Review Request 34751: CapScheduler Auto-Create and Cluster association

Posted by Erik Bergenholtz <eb...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34751/
-----------------------------------------------------------

(Updated May 28, 2015, 10:48 a.m.)


Review request for Ambari and Robert Levas.


Changes
-------

Added appropriate RAT check exclusions


Bugs: AMBARI-11436
    https://issues.apache.org/jira/browse/AMBARI-11436


Repository: ambari


Description
-------

Capacity Scheduler needs to support cluster association (configure based on cluster) and auto instance creation.


Diffs (updated)
-----

  contrib/views/capacity-scheduler/pom.xml 2712953 
  contrib/views/capacity-scheduler/readme.md e772508 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/ConfigurationService.java 8e1b6a6 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/Proxy.java c026b41 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/RequestBuilder.java 88e19bb 
  contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/proxy/ResponseTranslator.java 8e3b4a6 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/components/dropdownButtons.js cc15563 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js 9828399 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js b26697f 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 4dfb5e5 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less 82fbc07 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueContainer.hbs cdc9044 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/queues.hbs b412d7b 
  contrib/views/capacity-scheduler/src/main/resources/ui/test/integration/serializers_test.js PRE-CREATION 
  contrib/views/capacity-scheduler/src/main/resources/view.xml 2974f86 
  contrib/views/pom.xml 9fa698b 
  contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/AmbariApi.java 88e5f48 
  contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/URLStreamProviderBasicAuth.java 87a4acb 

Diff: https://reviews.apache.org/r/34751/diff/


Testing
-------

Local Unit Tests, deployment locally of built view to ensure that cluster association & auto-create works upon adding YARN service.


Thanks,

Erik Bergenholtz