[ 
https://issues.apache.org/jira/browse/ARTEMIS-4349?focusedWorklogId=878420&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-878420
 ]

ASF GitHub Bot logged work on ARTEMIS-4349:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 25/Aug/23 14:53
            Start Date: 25/Aug/23 14:53
    Worklog Time Spent: 10m 
      Work Description: gemmellr commented on code in PR #4584:
URL: https://github.com/apache/activemq-artemis/pull/4584#discussion_r1305770283


##########
artemis-server/src/main/java/org/apache/activemq/artemis/core/security/impl/SecurityStoreImpl.java:
##########
@@ -101,15 +103,15 @@ public SecurityStoreImpl(final 
HierarchicalRepository<Set<Role>> securityReposit
       if (authenticationCacheSize == 0) {
          authenticationCache = null;
       } else {
-         authenticationCache = CacheBuilder.newBuilder()
+         authenticationCache = Caffeine.newBuilder()
                                            
.maximumSize(authenticationCacheSize)
                                            
.expireAfterWrite(invalidationInterval, TimeUnit.MILLISECONDS)
                                            .build();

Review Comment:
   Fair. That was probably result of rebasing the changes again on top of your 
earlier PR, and those lines not having changes on them.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 878420)
    Time Spent: 9h 50m  (was: 9h 40m)

> Replace Guava cache with Caffeine
> ---------------------------------
>
>                 Key: ARTEMIS-4349
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-4349
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>    Affects Versions: 2.29.0
>            Reporter: Alexey Markevich
>            Priority: Major
>          Time Spent: 9h 50m
>  Remaining Estimate: 0h
>
> based on benchmark https://github.com/ben-manes/caffeine/wiki/Benchmarks



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to