Re: Review Request 49908: Storm Ambari View should provide a config page & make calls to Storm Rest API

2016-07-13 Thread Srimanth Gunturi

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


Ship it!




Ship It!

- Srimanth Gunturi


On July 13, 2016, 4:12 p.m., Sriharsha Chintalapani wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49908/
> ---
> 
> (Updated July 13, 2016, 4:12 p.m.)
> 
> 
> Review request for Ambari and Srimanth Gunturi.
> 
> 
> Bugs: AMBARI-17640
> https://issues.apache.org/jira/browse/AMBARI-17640
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Storm Ambari View should provide a config page & make calls to Storm Rest API
> 
> 
> Diffs
> -
> 
>   contrib/views/storm/src/main/java/org/apache/ambari/storm/ProxyServlet.java 
> 6e6bea2 
>   contrib/views/storm/src/main/resources/WEB-INF/web.xml e406de1 
>   contrib/views/storm/src/main/resources/scripts/components/BarChart.jsx 
> PRE-CREATION 
>   contrib/views/storm/src/main/resources/scripts/components/SearchLogs.jsx 
> PRE-CREATION 
>   contrib/views/storm/src/main/resources/scripts/components/SpoutGraph.jsx 
> 94bad11 
>   contrib/views/storm/src/main/resources/scripts/components/TopologyGraph.jsx 
> 1b1baf7 
>   contrib/views/storm/src/main/resources/scripts/models/VTopology.js da0a26d 
>   contrib/views/storm/src/main/resources/scripts/router/Router.js 1239b6f 
>   contrib/views/storm/src/main/resources/scripts/utils/Overrides.js 7bb6b3f 
>   contrib/views/storm/src/main/resources/scripts/utils/Utils.js 56a42a6 
>   
> contrib/views/storm/src/main/resources/scripts/views/ComponentDetailView.jsx 
> 5d77bdf 
>   contrib/views/storm/src/main/resources/scripts/views/TopologyDetailView.jsx 
> 7486948 
>   contrib/views/storm/src/main/resources/styles/style.css e4f5be2 
>   contrib/views/storm/src/main/resources/view.xml cbcf4ff 
> 
> Diff: https://reviews.apache.org/r/49908/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Sriharsha Chintalapani
> 
>



Re: Review Request 49908: Storm Ambari View should provide a config page & make calls to Storm Rest API

2016-07-13 Thread Sriharsha Chintalapani

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

(Updated July 13, 2016, 4:12 p.m.)


Review request for Ambari and Srimanth Gunturi.


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


Repository: ambari


Description
---

Storm Ambari View should provide a config page & make calls to Storm Rest API


Diffs
-

  contrib/views/storm/src/main/java/org/apache/ambari/storm/ProxyServlet.java 
6e6bea2 
  contrib/views/storm/src/main/resources/WEB-INF/web.xml e406de1 
  contrib/views/storm/src/main/resources/scripts/components/BarChart.jsx 
PRE-CREATION 
  contrib/views/storm/src/main/resources/scripts/components/SearchLogs.jsx 
PRE-CREATION 
  contrib/views/storm/src/main/resources/scripts/components/SpoutGraph.jsx 
94bad11 
  contrib/views/storm/src/main/resources/scripts/components/TopologyGraph.jsx 
1b1baf7 
  contrib/views/storm/src/main/resources/scripts/models/VTopology.js da0a26d 
  contrib/views/storm/src/main/resources/scripts/router/Router.js 1239b6f 
  contrib/views/storm/src/main/resources/scripts/utils/Overrides.js 7bb6b3f 
  contrib/views/storm/src/main/resources/scripts/utils/Utils.js 56a42a6 
  contrib/views/storm/src/main/resources/scripts/views/ComponentDetailView.jsx 
5d77bdf 
  contrib/views/storm/src/main/resources/scripts/views/TopologyDetailView.jsx 
7486948 
  contrib/views/storm/src/main/resources/styles/style.css e4f5be2 
  contrib/views/storm/src/main/resources/view.xml cbcf4ff 

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


Testing
---


Thanks,

Sriharsha Chintalapani