You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by unsleepy22 <gi...@git.apache.org> on 2016/05/25 03:41:09 UTC

[GitHub] storm pull request: [STORM-1277] port backtype.storm.daemon.execut...

GitHub user unsleepy22 opened a pull request:

    https://github.com/apache/storm/pull/1444

    [STORM-1277] port backtype.storm.daemon.executor to java

    Sorry for the long delay on this.
    
    This PR is not 100% completed yet, things remaining:
    1. a few tests will fail and I'm still working on this (basically with method replacements like local-executor/local-transfer-executor-tuple, which was originally executor/mk-executor-transfer-fn)
    2. my code base is behind master, so after all tests pass I will make a full merge from master
    Hopefully I can finish the problems by this week.
    
    Still, the base structure/design of the executor won't change, so I think I can get reviews from you first on the design.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/unsleepy22/storm storm-executor

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/1444.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1444
    
----
commit 3da8aa0c7c9854c82920ab25ea15859dc9a4658d
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-04-29T09:58:50Z

    init commit for executor

commit 69111064945ef7cad7706875739fbe0f1168f31e
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-11T09:29:25Z

    refined code, fixed bugs

commit e3cb37373f108055460dd40ef3d66aa306ef0b6e
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-11T13:18:49Z

    fixed bugs

commit 2f209adef20af5a43aac67c5420d838d8934b5a3
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-23T09:40:42Z

    added debug logs

commit b0efb563d7f0f3a5c99c18675871618dd1b52b20
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-23T13:37:33Z

    fixed type cast error

commit c1cfcc2ece8de6b51561b00ea1b39fe5d4f05c25
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-24T09:12:20Z

    messaging-test ok

commit ed2454d8429687d1fe2f7feb71b5e8fc87b434f3
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-24T10:52:30Z

    fixed tuple-test error

commit eab32b22e8cd08612b8db8cebf4f025df351eae4
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-24T11:26:21Z

    minor changes

commit 5883acf4d557c51e7002a5a4120e1c51d352c279
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-25T03:22:06Z

    changes for testing

commit c4ec8bacf38fb36bf2ec9ff352c21e7edef3297b
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-25T03:31:40Z

    revert changes to acker & testing

commit da454e90a3606b09b469f460a3e8f52104666162
Author: \u536b\u4e50 <we...@taobao.com>
Date:   2016-05-25T03:33:15Z

    revert changes to testing

----


---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request: [STORM-1277] port backtype.storm.daemon.execut...

Posted by unsleepy22 <gi...@git.apache.org>.
Github user unsleepy22 commented on the pull request:

    https://github.com/apache/storm/pull/1444#issuecomment-221465548
  
    sorry close and re-submit


---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request: [STORM-1277] port backtype.storm.daemon.execut...

Posted by unsleepy22 <gi...@git.apache.org>.
Github user unsleepy22 closed the pull request at:

    https://github.com/apache/storm/pull/1444


---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---