[#573] Migrate jcip to errorprone

Author: wburns

core/pom.xml

@@ -85,8 +85,9 @@
         </dependency>
 
         <dependency>
-            <groupId>net.jcip</groupId>
-            <artifactId>jcip-annotations</artifactId>
+            <groupId>com.google.errorprone</groupId>
+            <artifactId>error_prone_annotations</artifactId>
+            <scope>provided</scope>
             <optional>true</optional>
         </dependency>
 

core/src/main/java/module-info.java

@@ -3,7 +3,7 @@
    requires jdk.unsupported;
    requires com.fasterxml.jackson.core;
    requires static com.fasterxml.jackson.databind;
-   requires static jcip.annotations;
+   requires static com.google.errorprone.annotations;
    requires org.jboss.logging;
    requires static org.jboss.logging.annotations;
    opens org.infinispan.protostream;

core/src/main/java/org/infinispan/protostream/SerializationContext.java

@@ -2,16 +2,13 @@
 
 import java.util.Set;
 
-import net.jcip.annotations.ThreadSafe;
-
 /**
  * A repository for Protobuf type definitions and their marshallers. All ProtoStream marshalling operations happen in
  * the context of a {@code SerializationContext}.
  *
  * @author anistor@redhat.com
  * @since 1.0
  */
-@ThreadSafe
 public interface SerializationContext extends ImmutableSerializationContext {
 
    /**

core/src/main/java/org/infinispan/protostream/impl/RandomAccessOutputStreamImpl.java

@@ -8,9 +8,6 @@
 
 import org.infinispan.protostream.RandomAccessOutputStream;
 
-import net.jcip.annotations.NotThreadSafe;
-
-@NotThreadSafe
 public class RandomAccessOutputStreamImpl extends OutputStream implements RandomAccessOutputStream {
 
    static final int MIN_SIZE = 32;

core/src/main/java/org/infinispan/protostream/impl/SerializationContextImpl.java

@@ -27,7 +27,7 @@
 import org.infinispan.protostream.descriptors.ResolutionContext;
 import org.infinispan.protostream.impl.parser.ProtostreamProtoParser;
 
-import net.jcip.annotations.GuardedBy;
+import com.google.errorprone.annotations.concurrent.GuardedBy;
 
 /**
  * @author anistor@redhat.com

parent/pom.xml

@@ -80,7 +80,6 @@
         <version.junit>4.13.2</version.junit>
         <version.compile-testing>0.21</version.compile-testing>
         <version.mockito>5.20.0</version.mockito>
-        <version.jcipannotations>1.0</version.jcipannotations>
         <version.jboss.logging>3.6.1.Final</version.jboss.logging>
         <version.jboss.logging.annotations>3.0.4.Final</version.jboss.logging.annotations>
         <version.jboss.logging.processor>3.0.4.Final</version.jboss.logging.processor>
@@ -89,6 +88,7 @@
         <version.jackson>2.20.1</version.jackson>
         <version.auto-service>1.1.1</version.auto-service>
         <version.assertj>3.27.6</version.assertj>
+        <version.errorprone>2.23.0</version.errorprone>
         <version.protobuf>4.33.0</version.protobuf>
     </properties>
 
@@ -203,9 +203,9 @@
             </dependency>
 
             <dependency>
-                <groupId>net.jcip</groupId>
-                <artifactId>jcip-annotations</artifactId>
-                <version>${version.jcipannotations}</version>
+                <groupId>com.google.errorprone</groupId>
+                <artifactId>error_prone_annotations</artifactId>
+                <version>${version.errorprone}</version>
                 <scope>provided</scope>
                 <optional>true</optional>
             </dependency>