[jira] [Commented] (AMQ-5876) improve performance of TextFileCertificateLoginModule when many entries are in the "textfiledn.user " file

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

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

ASF subversion and git services commented on AMQ-5876:
------------------------------------------------------

Commit 8d63083dfff064efadbe99a66deeedef829e519e in activemq's branch refs/heads/master from [~gtully]
[ https://git-wip-us.apache.org/repos/asf?p=activemq.git;h=8d63083 ]

https://issues.apache.org/jira/browse/AMQ-5876 - load groups properties into per user set once also to avoid parse per login attempt


> improve performance of TextFileCertificateLoginModule when many entries are in the "textfiledn.user " file
> ----------------------------------------------------------------------------------------------------------
>
>                 Key: AMQ-5876
>                 URL: https://issues.apache.org/jira/browse/AMQ-5876
>             Project: ActiveMQ
>          Issue Type: Improvement
>          Components: jaas, security
>    Affects Versions: 5.11.1
>            Reporter: Gary Tully
>            Assignee: Gary Tully
>              Labels: jaas
>             Fix For: 5.12.0
>
>
> With a large number of entries ( 200,000 ) in the "org.apache.activemq.jaas.textfiledn.user" file the performance seemed to degrade. 
> To demonstrate the performance difference;
> {code}
> 1) for 100 entries and calling initialize,login(),commit 10 times - Time taken is 73 miliseconds
> 2) for 200,000 entries and calling initialize,login(),commit 10 times - Time taken is  5020 miliseconds
> {code}
> Suggested improvements:
> - avoid loading the  org.apache.activemq.jaas.textfiledn.user file each time - in PropertiesLoginModule.java, the file is only read when it changes, using the file modification time.
> - avoid iterating through the Properties object, using a Map instead to retrieve the userName



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)