[jira] [Commented] (STORM-947) replace all `backtype.storm.scheduler.ExecutorDetails` with `backtype.storm.generated.ExecutorInfo `

[ASF GitHub Bot (JIRA)]

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

ASF GitHub Bot commented on STORM-947:
--------------------------------------

Github user chuanlei commented on a diff in the pull request:

    https://github.com/apache/storm/pull/640#discussion_r34878435
  
    --- Diff: storm-core/src/clj/backtype/storm/scheduler/DefaultScheduler.clj 
---
    @@ -18,7 +18,7 @@
       (:require [backtype.storm.scheduler.EvenScheduler :as EvenScheduler])
       (:import [backtype.storm.scheduler IScheduler Topologies
                 Cluster TopologyDetails WorkerSlot SchedulerAssignment
    -            EvenScheduler ExecutorDetails])
    +            EvenScheduler])
    --- End diff --
    
    So, is the original import unnecessary?


> 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)