[GitHub] storm pull request: Update get-task-object function, change the ...

2015-03-03 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/433#issuecomment-76966697 @d2r I haved imported backtype.storm.generated.StormTopology in task.clj. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] storm pull request: Update get-task-object function, change the ...

2015-03-02 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/312#issuecomment-76868454 @d2r ok, i have closed this pull request. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] storm pull request: Update get-task-object function, change the ...

2015-03-02 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/433#issuecomment-76870913 @d2r I have added [STORM-554] to the title of this pull request and closed #312. --- If your project is set up for it, you can reply to this email and have your

[GitHub] storm pull request: Update get-task-object function, change the ...

2015-03-01 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/312#issuecomment-76647927 @d2r I create a new repository, and pull this request again. see #433. I have filed a JIRA, https://issues.apache.org/jira/browse/STORM-554. you can merge #433

[GitHub] storm pull request: Update get-task-object function, change the ...

2015-02-15 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/312#issuecomment-74463198 @revans2 I create a new repository, and pull this request again. see https://github.com/apache/storm/pull/433. I have filed a JIRA, https://issues.apache.org/jira

[GitHub] storm pull request: update worker.clj-delete missing-tasks check...

2015-02-15 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/288#issuecomment-74460727 @revans2 ok, i will close this pull request. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] storm pull request: update worker.clj-delete missing-tasks check...

2015-02-15 Thread BuDongDong
Github user BuDongDong closed the pull request at: https://github.com/apache/storm/pull/288 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] storm pull request: Update get-task-object function, change the ...

2015-02-15 Thread BuDongDong
GitHub user BuDongDong opened a pull request: https://github.com/apache/storm/pull/433 Update get-task-object function, change the type of first param topology from ^TopologyContext to ^StormTopology Update get-task-object function, change the type of first param topology from

[GitHub] storm pull request: update worker.clj-delete missing-tasks check...

2015-01-12 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/288#issuecomment-69576596 @revans2 I have filed a JIRA(STORM-527) for this, please see https://issues.apache.org/jira/browse/STORM-527 --- If your project is set up for it, you can reply

[GitHub] storm pull request: update worker.clj-delete missing-tasks check...

2015-01-12 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/288#issuecomment-69574937 @revans2 thank you --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] storm pull request: Update stats.clj rolling-window-set function...

2014-12-15 Thread BuDongDong
GitHub user BuDongDong opened a pull request: https://github.com/apache/storm/pull/348 Update stats.clj rolling-window-set function, exchange the real argume... ...nt num-buckets and s of rolling-window function (defn rolling-window-set [updater merger extractor num-buckets

[GitHub] storm pull request: Update stats.clj rolling-window-set function...

2014-12-15 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/348#issuecomment-66964691 (defrecord RollingWindow [updater merger extractor bucket-size-secs num-buckets buckets]) I think that the bucket-size-secs of RollingWindow is 20

[GitHub] storm pull request: Update stats.clj rolling-window-set function...

2014-12-15 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/348#issuecomment-67004693 @nathanmarz Could you check this bug, thank you very much --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] storm pull request: Update stats.clj rolling-window-set function...

2014-12-15 Thread BuDongDong
Github user BuDongDong closed the pull request at: https://github.com/apache/storm/pull/348 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] storm pull request: Update get-task-object function, change the ...

2014-11-15 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/312#issuecomment-63206771 @harshach the get-task-object is called by mk-task-data function in task.clj, mk-task-data is defined as following: (defn mk-task-data [executor-data task-id

[GitHub] storm pull request: Update get-task-object function, change the ...

2014-11-12 Thread BuDongDong
Github user BuDongDong commented on the pull request: https://github.com/apache/storm/pull/312#issuecomment-62844422 (defn- get-task-object [^TopologyContext topology component-id] ... ... ) i think the type of param topology is StormTopology not TopologyContext

[GitHub] storm pull request: Update get-task-object function, change the ...

2014-11-12 Thread BuDongDong
GitHub user BuDongDong opened a pull request: https://github.com/apache/storm/pull/312 Update get-task-object function, change the type of first param topology from ^TopologyContext to ^StormTopology update get-task-object function, change the type of first param topology from

[GitHub] storm pull request: update worker.clj-delete missing-tasks check...

2014-10-09 Thread BuDongDong
GitHub user BuDongDong opened a pull request: https://github.com/apache/storm/pull/288 update worker.clj-delete missing-tasks checking missing-tasks set is created by two times filter my-assignment map, so i think keys(my-assignment) contains missing-tasks set. missing-tasks