[
https://issues.apache.org/jira/browse/STORM-1272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15142801#comment-15142801
]
ASF GitHub Bot commented on STORM-1272:
---------------------------------------
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/1095#issuecomment-182889339
The changes look good to me and the unit tests all pass. My only concern
now is the performance, and the disruptor queue code is on the critical path.
Have you run any performance tests? If not I can run some to do a quick
comparison.
> 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)
