[
https://issues.apache.org/jira/browse/STORM-1248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15143310#comment-15143310
]
ASF GitHub Bot commented on STORM-1248:
---------------------------------------
Github user knusbaum commented on a diff in the pull request:
https://github.com/apache/storm/pull/1097#discussion_r52651786
--- Diff: storm-core/src/clj/org/apache/storm/daemon/worker.clj ---
@@ -461,11 +460,11 @@
)))))
(defn register-callbacks [worker]
- (log-message "Registering IConnectionCallbacks for " (:assignment-id
worker) ":" (:port worker))
- (msg-loader/register-callback (:transfer-local-fn worker)
- (:receiver worker)
- (:storm-conf worker)
- (worker-context worker)))
+ (let [transfer-local-fn (:transfer-local-fn worker) ^IConnection socket
(:receiver worker)]
--- End diff --
`let` bindings should be one per line.
> port backtype.storm.messaging.loader to java
> ---------------------------------------------
>
> Key: STORM-1248
> URL: https://issues.apache.org/jira/browse/STORM-1248
> Project: Apache Storm
> Issue Type: New Feature
> Components: storm-core
> Reporter: Robert Joseph Evans
> Assignee: Abhishek Agarwal
> Labels: java-migration, jstorm-merger
>
> Just a small bit of glue code for calling transfer-local-fn when messaging
> batch arrives.
> In the short term we may need to use an IFn, until the transfer-local-fn is
> migrated to java too.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
