Add missing @OverRide annotations (CodeQL java/missing-override-annotation) (#246)

Annotate 308 methods that override a superclass method or implement an
interface method but were missing @OverRide, across 97 files in 9 modules.
Annotation-only and behavior-preserving; verified by compiling every
affected Maven module (plus javac for the two persistit Ant examples).

Author: vharseko

bloomfilter/core/src/main/java/org/forgerock/bloomfilter/BloomFilters.java

@@ -12,6 +12,7 @@
  * information: "Portions copyright [year] [name of copyright owner]".
  *
  * Copyright 2015 ForgeRock AS.
+ * Portions Copyrighted 2026 3A Systems, LLC
  */
 
 package org.forgerock.bloomfilter;
@@ -297,6 +298,7 @@ public RollingBloomFilterBuilder(final BloomFilterBuilder<T> toCopy) {
         }
 
         @VisibleForTesting
+        @Override
         RollingBloomFilterBuilder<T> withClock(final TimeService clock) {
             Reject.ifNull(clock);
             this.clock = clock;

build-tools/src/main/java/org/forgerock/testng/ForgeRockTestListener.java

@@ -13,6 +13,7 @@
  *
  *      Copyright 2008 Sun Microsystems, Inc.
  *      Portions copyright 2011-2012 ForgeRock AS
+ * Portions Copyrighted 2026 3A Systems, LLC
  */
 package org.forgerock.testng;
 
@@ -302,6 +303,7 @@ public ForgeRockTestListener() {
         initializeProgressVars();
     }
 
+    @Override
     public void generateReport(final List<XmlSuite> xmlSuites, final List<ISuite> suites,
             final String outputDirectory) {
         final File reportFile = new File(outputDirectory, REPORT_FILE_NAME);
@@ -310,6 +312,7 @@ public void generateReport(final List<XmlSuite> xmlSuites, final List<ISuite> su
         writeAntTestsFailedMarker(outputDirectory);
     }
 
+    @Override
     public void onConfigurationFailure(final ITestResult tr) {
         super.onConfigurationFailure(tr);
 
@@ -331,18 +334,21 @@ public void onConfigurationFailure(final ITestResult tr) {
         _bufferedTestFailures.append(failureInfo);
     }
 
+    @Override
     public void onStart(final ITestContext testContext) {
         super.onStart(testContext);
 
         // Delete the previous report if it's there.
         new File(testContext.getOutputDirectory(), REPORT_FILE_NAME).delete();
     }
 
+    @Override
     public void onTestFailedButWithinSuccessPercentage(final ITestResult tr) {
         super.onTestFailedButWithinSuccessPercentage(tr);
         onTestFinished(tr);
     }
 
+    @Override
     public void onTestFailure(final ITestResult tr) {
         super.onTestFailure(tr);
 
@@ -378,18 +384,21 @@ public void onTestFailure(final ITestResult tr) {
         onTestFinished(tr);
     }
 
+    @Override
     public void onTestSkipped(final ITestResult tr) {
         super.onTestSkipped(tr);
         onTestFinished(tr);
     }
 
+    @Override
     public void onTestStart(final ITestResult tr) {
         super.onTestStart(tr);
         enforceTestClassTypeAndAnnotations(tr);
         checkForInterleavedBetweenClasses(tr);
         enforceMethodHasAnnotation(tr);
     }
 
+    @Override
     public void onTestSuccess(final ITestResult tr) {
         super.onTestSuccess(tr);
         onTestFinished(tr);
@@ -584,6 +593,7 @@ private List<TestClassResults> getClassesDescendingSortedByDuration() {
         final List<TestClassResults> allClasses = new ArrayList<TestClassResults>(
                 _classResults.values());
         Collections.sort(allClasses, new Comparator<TestClassResults>() {
+            @Override
             public int compare(final TestClassResults o1, final TestClassResults o2) {
                 if (o1._totalDurationMs > o2._totalDurationMs) {
                     return -1;
@@ -607,6 +617,7 @@ private String getFqMethod(final ITestResult result) {
     private List<TestMethodResults> getMethodsDescendingSortedByDuration() {
         final List<TestMethodResults> allMethods = getAllMethodResults();
         Collections.sort(allMethods, new Comparator<TestMethodResults>() {
+            @Override
             public int compare(final TestMethodResults o1, final TestMethodResults o2) {
                 if (o1._totalDurationMs > o2._totalDurationMs) {
                     return -1;

cassandra-embedded/src/main/java/org/openidentityplatform/commons/cassandra/EmbeddedServer.java

@@ -47,6 +47,7 @@ public class EmbeddedServer implements Runnable, AutoCloseable {
 	final private ExecutorService executor = Executors.newSingleThreadExecutor();
 	private CassandraDaemon cassandraDaemon;
 
+	@Override
 	public void run() {
 		try {
 			//check for external cassandra settings
@@ -139,6 +140,7 @@ public void run() {
 		}
 	}
 
+	@Override
 	public void close() {
 		if (cassandraDaemon!=null) {
 			cassandraDaemon.stop();

commons/audit/handler-csv/src/main/java/org/forgerock/audit/handlers/csv/CsvWriter.java

@@ -48,5 +48,6 @@ interface CsvWriter extends AutoCloseable {
      */
     void flush() throws IOException;
 
+    @Override
     void close() throws IOException;
 }

commons/audit/handler-csv/src/main/java/org/forgerock/audit/handlers/csv/StandardCsvWriter.java

@@ -135,6 +135,7 @@ public void writeEvent(Map<String, String> values) throws IOException {
      * Flush the data into the CSV file.
      * @throws IOException if an I/O error occurs while flushing.
      */
+    @Override
     public void flush() throws IOException {
         csvWriter.flush();
     }

commons/audit/handler-jdbc/src/main/java/org/forgerock/audit/handlers/jdbc/BufferedJdbcAuditEventExecutor.java

@@ -13,6 +13,7 @@
  *
  * Copyright 2015-2016 ForgeRock AS.
  * Portions Copyright 2016 Nomura Research Institute, Ltd.
+ * Portions Copyrighted 2026 3A Systems, LLC
  */
 package org.forgerock.audit.handlers.jdbc;
 
@@ -87,6 +88,7 @@ public BufferedJdbcAuditEventExecutor(int capacity, boolean autoFlush, JdbcAudit
         this.maxBatchedEvents = maxBatchedEvents;
     }
 
+    @Override
     public void flush() {
         try {
             while (!queue.isEmpty()) {

commons/audit/handler-jms/src/main/java/org/forgerock/audit/handlers/jms/JmsAuditEventHandler.java

@@ -119,6 +119,7 @@ public void shutdown() throws ResourceException {
      * @param auditEvent The event to convert to a JMS TextMessage and publish on the JMS Topic.
      * @return a promise with either a response or an exception
      */
+    @Override
     public Promise<ResourceResponse, ResourceException> publishEvent(Context context, String auditTopic,
             JsonValue auditEvent) {
         try {

commons/audit/handler-jms/src/main/java/org/forgerock/audit/handlers/jms/JndiJmsContextManager.java

@@ -12,7 +12,7 @@
  * information: "Portions copyright [year] [name of copyright owner]".
  *
  * Copyright 2016 ForgeRock AS.
- * Portions copyright 2024 3A Systems LLC.
+ * Portions copyright 2024-2026 3A Systems LLC.
  */
 
 package org.forgerock.audit.handlers.jms;
@@ -67,6 +67,7 @@ class JndiJmsContextManager implements JmsContextManager {
      * @return The {@link Topic JMS topic} to use for JMS publish/subscribe functionality.
      * @throws InternalServerErrorException If unable to retrieve the {@link Topic JMS topic}.
      */
+    @Override
     public Topic getTopic() throws InternalServerErrorException {
         try {
             if (topic == null) {
@@ -83,6 +84,7 @@ public Topic getTopic() throws InternalServerErrorException {
      * @return the {@link ConnectionFactory JMS connection factory} to use to connect to JMS services.
      * @throws InternalServerErrorException If unable to retrieve the {@link ConnectionFactory JMS connection factory}.
      */
+    @Override
     public ConnectionFactory getConnectionFactory() throws InternalServerErrorException {
         try {
             if (connectionFactory == null) {

commons/audit/handler-syslog/src/main/java/org/forgerock/audit/handlers/syslog/SyslogConnection.java

@@ -13,6 +13,7 @@
  *
  * Copyright 2013 Cybernetica AS
  * Portions copyright 2014-2015 ForgeRock AS.
+ * Portions Copyrighted 2026 3A Systems, LLC
  */
 package org.forgerock.audit.handlers.syslog;
 
@@ -29,5 +30,6 @@ interface SyslogConnection extends AutoCloseable {
 
     void flush() throws IOException;
 
+    @Override
     void close();
 }

commons/audit/handler-syslog/src/main/java/org/forgerock/audit/handlers/syslog/SyslogPublisher.java

@@ -13,6 +13,7 @@
  *
  * Copyright 2013 Cybernetica AS
  * Portions copyright 2014-2015 ForgeRock AS.
+ * Portions Copyrighted 2026 3A Systems, LLC
  */
 package org.forgerock.audit.handlers.syslog;
 
@@ -35,5 +36,6 @@ interface SyslogPublisher extends AutoCloseable {
     /**
      * Closes the underlying connection.
      */
+    @Override
     void close();
 }