[
https://issues.apache.org/jira/browse/STORM-947?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14631157#comment-14631157
]
ASF GitHub Bot commented on STORM-947:
--------------------------------------
Github user chuanlei commented on the pull request:
https://github.com/apache/storm/pull/640#issuecomment-122239340
I think it is good since `ExecutorDetails` does not take more information.
And this change make Storm code more concise.
But I'm sure whether `ExecutorDetails` is designed for extension in the
future.
> replace all `backtype.storm.scheduler.ExecutorDetails` with
> `backtype.storm.generated.ExecutorInfo `
> ------------------------------------------------------------------------------------------------------
>
> Key: STORM-947
> URL: https://issues.apache.org/jira/browse/STORM-947
> Project: Apache Storm
> Issue Type: Sub-task
> Affects Versions: 0.11.0
> Reporter: caofangkun
> Assignee: caofangkun
> Priority: Minor
>
> replace all
> {code}
> backtype.storm.scheduler.ExecutorDetails
> {code}
> with
> {code}
> backtype.storm.generated.ExecutorInfo
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
