[jira] Commented: (AMQ-1231) Performance issues with org/apache/activemq/broker/region/PrefetchSubscription.java and org/apache/activemq/store/kahadaptor/KahaTopicMessageStore.java

["Kevin Yaussy (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/activemq/browse/AMQ-1231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=41218#action_41218 ] 

Kevin Yaussy commented on AMQ-1231:
-----------------------------------

Rob, sorry for the delay in responding.  I've taken the 1/17/2008 snapshot and have been testing with that for the last couple weeks.  Base Kaha performance actually seems better, anyway. So, I think what you've got is good.

Thanks.  

Kevin.


> Performance issues with org/apache/activemq/broker/region/PrefetchSubscription.java and org/apache/activemq/store/kahadaptor/KahaTopicMessageStore.java
> -------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: AMQ-1231
>                 URL: https://issues.apache.org/activemq/browse/AMQ-1231
>             Project: ActiveMQ
>          Issue Type: Improvement
>          Components: Broker
>    Affects Versions: 5.0.0
>            Reporter: Kevin Yaussy
>            Assignee: Rob Davies
>             Fix For: 5.1.0
>
>         Attachments: KahaTopicMessageStore.java, PrefetchSubscription.java
>
>
> Using Solaris 10 DTrace, I found a couple intense synchronization pinch-points in the Broker.  Without altering the locking semantics, I converted the synchronized methods and one synchronized code block to the JDK 1.5 ReentrantLock.  There was a pretty significant performance benefit with this change.  I've attached the code, together with the old synchronized code commented out for ease of change examination.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.