[
https://issues.apache.org/jira/browse/STORM-1248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15143422#comment-15143422
]
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_r52660183
--- Diff: storm-core/src/clj/org/apache/storm/daemon/worker.clj ---
@@ -487,11 +486,12 @@
)))))
(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)]
+ (log-message "Registering IConnectionCallbacks for " (:assignment-id
worker) ":" (:port worker))
+ (.registerRecv socket (DeserializingConnectionCallback. (:storm-conf
worker)
--- End diff --
```
(reify ThreadLocal
(initialValue [this] (KryoTupleDeserializer. (:storm-conf worker)
(worker-context worker)))
```
That's it for the thread-local.
> 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)
