[
https://issues.apache.org/jira/browse/METRON-982?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16037024#comment-16037024
]
ASF GitHub Bot commented on METRON-982:
---------------------------------------
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/607#discussion_r120117733
--- Diff:
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/StormStatusServiceImpl.java
---
@@ -49,6 +44,11 @@ public StormStatusServiceImpl(Environment environment,
RestTemplate restTemplate
}
@Override
+ public SupervisorSummary getSupervisorSummary(){
+ return restTemplate.getForObject("http://"; +
environment.getProperty(STORM_UI_SPRING_PROPERTY) + SUPERVISOR_SUMMARY_URL,
SupervisorSummary.class);
--- End diff --
I was just following the convention of the service as I found it. I think
that is a separate issue as @simonellistonball points out
> REST STORM Support for supervisor summary
> -----------------------------------------
>
> Key: METRON-982
> URL: https://issues.apache.org/jira/browse/METRON-982
> Project: Metron
> Issue Type: Improvement
> Reporter: Otto Fowler
> Assignee: Otto Fowler
>
> The supervisor summary can be used for information on the cluster including
> the number of slots etc
> StormStatusServiceImpl already calls the storm ui rest endpoints and should
> be extended to call:
> {code}
> /api/v1/supervisor/summary (GET)
> returns all supervisors summary
> Response Fields:
> Field Value Description
> id String Supervisor's id
> host String Supervisor's host name
> uptime String Shows how long the supervisor is running
> slotsTotal Integer Total number of available worker slots for this
> supervisor
> slotsUsed Integer Number of worker slots used on this supervisor
> Sample Response:
> json { "supervisors": [ { "id": "0b879808-2a26-442b-8f7d-23101e0c3696",
> "host": "10.11.1.7", "uptime": "5m 58s", "slotsTotal": 4, "slotsUsed": 3 } ] }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
