[ https://issues.apache.org/jira/browse/STORM-2026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15412748#comment-15412748 ]
ASF GitHub Bot commented on STORM-2026: --------------------------------------- Github user satishd commented on a diff in the pull request: https://github.com/apache/storm/pull/1616#discussion_r73982987 --- Diff: storm-core/src/jvm/org/apache/storm/executor/Executor.java --- @@ -204,8 +204,8 @@ public static Executor mkExecutor(Map workerData, List<Long> executorId, Map<Str throw Utils.wrapInRuntime(ex); } } - executor.init(idToTask); + executor.idToTask = idToTask; --- End diff -- @HeartSaVioR Sorry, missed that. This code looks convoluted because it is ported from clojure. I am fine with that for now as these PRs are more about poring from clojure to Java. We can address these later. > Inconsistency between (SpoutExecutor, BoltExecutor) and (spout-transfer-fn, > bolt-transfer-fn) > --------------------------------------------------------------------------------------------- > > Key: STORM-2026 > URL: https://issues.apache.org/jira/browse/STORM-2026 > Project: Apache Storm > Issue Type: Bug > Components: storm-core > Affects Versions: 2.0.0 > Reporter: Jungtaek Lim > Assignee: Jungtaek Lim > > As I left the comment from > https://github.com/apache/storm/pull/1445#discussion_r73255197 for pull > request, there's some difference between SpoutExecutor / BoltExecutor and > spout-transfer-fn / bolt-transfer-fn. > While it's not that big to fix, I just want to not block port work and just > address from here. -- This message was sent by Atlassian JIRA (v6.3.4#6332)