[
https://issues.apache.org/jira/browse/STORM-1272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141811#comment-15141811
]
ASF GitHub Bot commented on STORM-1272:
---------------------------------------
Github user knusbaum commented on a diff in the pull request:
https://github.com/apache/storm/pull/1095#discussion_r52535123
--- Diff: storm-core/src/clj/org/apache/storm/daemon/executor.clj ---
@@ -227,13 +228,13 @@
component-id (.getComponentId worker-context (first task-ids))
storm-conf (normalized-component-conf (:storm-conf worker)
worker-context component-id)
executor-type (executor-type worker-context component-id)
- batch-transfer->worker (disruptor/disruptor-queue
+ batch-transfer->worker (DisruptorQueue.
--- End diff --
Do we want to rename this? Usually the `->` implies a map.
> port backtype.storm.disruptor to java
> -------------------------------------
>
> Key: STORM-1272
> URL: https://issues.apache.org/jira/browse/STORM-1272
> Project: Apache Storm
> Issue Type: New Feature
> Components: storm-core
> Reporter: Robert Joseph Evans
> Assignee: Abhishek Agarwal
> Labels: java-migration, jstorm-merger
>
> wrapper around the disruptor queue. Might need some abstract base classes if
> it makes since.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
