GitHub user unsleepy22 opened a pull request:
https://github.com/apache/storm/pull/1445
[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 fail and I'm still working on this (basically with method
replacement of `local-executor/local-transfer-executor-tuple`, which was
originally `executor/mk-executor-transfer-fn` in testing.clj, I'm not familiar
with this and still trying to figure out how, appreciate any help)
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 guys 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/1445.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 #1445
----
commit b13ac017938c3a55181d33faf77570be0d2e001e
Author: å«ä¹ <[email protected]>
Date: 2016-04-29T09:58:50Z
port executor to java
things remaining:
1. a few tests to be fixed
2. need to upmerge with master
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
