[ 
https://issues.apache.org/jira/browse/SLIDER-778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14309507#comment-14309507
 ] 

Steve Loughran commented on SLIDER-778:
---------------------------------------

main bit of work here is that instead of refreshing and retrieving cluster 
Status to get the map of roles->containers, but instead exposing the underlying 
appstate operation, and making that operation faster. It's more efficient now 
(no ser/deser in the same JVM), and guaranteed to be up to date. 

Mock tests altered to work with new operation

> Clean up AgentProviderService slightly
> --------------------------------------
>
>                 Key: SLIDER-778
>                 URL: https://issues.apache.org/jira/browse/SLIDER-778
>             Project: Slider
>          Issue Type: Bug
>          Components: agent-provider
>    Affects Versions: Slider 0.70
>            Reporter: Steve Loughran
>            Assignee: Steve Loughran
>            Priority: Minor
>             Fix For: Slider 0.70
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Clean up the agent provider slightly 
> # do some IDE logging improvements hints
> # factor out a recurrent check of maps & lists being non-null & non-empty
> # replace the (inefficient) getClusterState->getClusterNodes logic with 
> something which achieves the same goals with less structures being built per 
> call



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to