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

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

                Author: ASF GitHub Bot
            Created on: 07/Jul/23 17:37
            Start Date: 07/Jul/23 17:37
    Worklog Time Spent: 10m 
      Work Description: gemmellr commented on code in PR #4540:
URL: https://github.com/apache/activemq-artemis/pull/4540#discussion_r1256183637


##########
pom.xml:
##########
@@ -1097,6 +1094,13 @@
                               <arg>-XDcompilePolicy=simple</arg>
                               <arg>-Xplugin:ErrorProne 
-Xep:ThreadLocalUsage:ERROR -Xep:MissingOverride:ERROR 
-Xep:NonAtomicVolatileUpdate:ERROR -Xep:SynchronizeOnNonFinalField:ERROR 
-Xep:StaticQualifiedUsingExpression:ERROR -Xep:WaitNotInLoop:ERROR 
-Xep:BanJNDI:OFF -XepExcludedPaths:.*/generated-sources/.*</arg>
                           </compilerArgs>
+                          <annotationProcessorPaths>
+                              <path>
+                                  <groupId>com.google.errorprone</groupId>
+                                  <artifactId>error_prone_core</artifactId>
+                                  <version>${errorprone.version}</version>
+                              </path>
+                          </annotationProcessorPaths>

Review Comment:
   >[INFO] +- org.apache.curator:curator-client:jar:5.2.0:compile
   [INFO] |  \- com.google.guava:guava:jar:27.0.1-jre:compile
   
   So that module is now using Guava 27.0.1 (instead of 32.0.1 as before), 
which presumably doesn't have the thing needed by ErrorProne, which normally 
uses a newer version.
   
   Maven is picking the older version during resolution, probably because it is 
earlier on the dependency set than the ErrorProne dep is (it will be last due 
to being in a profile). Its doing that instead of just using the specified 
dependencyManagement guava version that was removed from the build. Restoring 
that may fix things. Alternative perhaps there is a newer curator-client 
version that uses a recent guava.





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

    Worklog Id:     (was: 869854)
    Time Spent: 4h 10m  (was: 4h)

> 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: 4h 10m
>  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