Initial apply of OpenReWrite recipe

```
mvn -U org.openrewrite.maven:rewrite-maven-plugin:run \
 -Drewrite.recipeArtifactCoordinates=org.openrewrite.recipe:rewrite-migrate-java:RELEASE \
 -Drewrite.activeRecipes=org.openrewrite.java.migrate.UpgradeToJava17 \
 -Drewrite.exportDatatables=true
```

Author: groldan

geowebcache/arcgiscache/src/test/java/org/geowebcache/arcgis/layer/XMLConfigurationLayerConformanceWithArcGisLayersTest.java

@@ -22,7 +22,7 @@
 import java.net.URISyntaxException;
 import java.net.URL;
 import java.nio.file.Files;
-import java.nio.file.Paths;
+import java.nio.file.Path;
 import java.util.List;
 import java.util.Optional;
 import java.util.stream.Collectors;
@@ -123,8 +123,8 @@ protected Matcher<TileLayer> infoEquals(TileLayer expected) {
 
             @Override
             public boolean matches(Object item) {
-                return item instanceof ArcGISCacheLayer
-                        && ((ArcGISCacheLayer) item).getBackendTimeout().equals(expected.getBackendTimeout());
+                return item instanceof ArcGISCacheLayer agiscl
+                        && agiscl.getBackendTimeout().equals(expected.getBackendTimeout());
             }
         };
     }
@@ -135,8 +135,8 @@ protected Matcher<TileLayer> infoEquals(int expected) {
 
             @Override
             public boolean matches(Object item) {
-                return item instanceof ArcGISCacheLayer
-                        && ((ArcGISCacheLayer) item).getBackendTimeout().equals(expected);
+                return item instanceof ArcGISCacheLayer agiscl
+                        && agiscl.getBackendTimeout().equals(expected);
             }
         };
     }
@@ -158,7 +158,7 @@ protected void makeConfigFile() throws Exception {
             configFile = temp.newFile("geowebcache.xml");
 
             URL source = XMLConfigurationLayerConformanceWithArcGisLayersTest.class.getResource("geowebcache.xml");
-            try (Stream<String> lines = Files.lines(Paths.get(source.toURI()))) {
+            try (Stream<String> lines = Files.lines(Path.of(source.toURI()))) {
                 List<String> replaced = lines.map(line -> {
                             String tilingSchemePath =
                                     resourceAsFile("/compactcache/Conf.xml").getAbsolutePath();

geowebcache/azureblob/pom.xml

@@ -37,6 +37,12 @@
       <artifactId>gwc-core</artifactId>
       <version>${project.version}</version>
     </dependency>
+  <dependency>
+   <groupId>jakarta.annotation</groupId>
+   <artifactId>jakarta.annotation-api</artifactId>
+   <version>1.3.5</version>
+   <scope>provided</scope>
+  </dependency>
 
     <dependency>
       <groupId>com.azure</groupId>

geowebcache/azureblob/src/main/java/org/geowebcache/azure/AzureBlobStoreData.java

@@ -160,9 +160,9 @@ public String getLocation() {
         String container = this.getContainer();
         String prefix = this.getPrefix();
         if (prefix == null) {
-            return String.format("container: %s", container);
+            return "container: %s".formatted(container);
         } else {
-            return String.format("container: %s prefix: %s", container, prefix);
+            return "container: %s prefix: %s".formatted(container, prefix);
         }
     }
 

geowebcache/azureblob/src/main/java/org/geowebcache/azure/AzureBlobStoreInfo.java

@@ -16,6 +16,7 @@
 import static com.google.common.base.Preconditions.checkNotNull;
 import static com.google.common.base.Preconditions.checkState;
 
+import java.io.Serial;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 import javax.annotation.Nullable;
@@ -30,6 +31,7 @@
 
 /** Plain old java object representing the configuration for an Azure blob store. */
 public class AzureBlobStoreInfo extends BlobStoreInfo {
+    @Serial
     private static final long serialVersionUID = -8068069256598987874L;
 
     /**
@@ -229,9 +231,9 @@ public String getLocation() {
         String container = this.getContainer();
         String prefix = this.getPrefix();
         if (prefix == null) {
-            return String.format("container: %s", container);
+            return "container: %s".formatted(container);
         } else {
-            return String.format("container: %s prefix: %s", container, prefix);
+            return "container: %s prefix: %s".formatted(container, prefix);
         }
     }
 

geowebcache/azureblob/src/main/java/org/geowebcache/azure/AzureClient.java

@@ -142,7 +142,7 @@ String getServiceURL(AzureBlobStoreData configuration) {
             // default to account name based location
             String proto = configuration.isUseHTTPS() ? "https" : "http";
             String account = configuration.getAccountName();
-            serviceURL = String.format("%s://%s.blob.core.windows.net", proto, account);
+            serviceURL = "%s://%s.blob.core.windows.net".formatted(proto, account);
         }
         return serviceURL;
     }

geowebcache/azureblob/src/main/java/org/geowebcache/azure/DeleteManager.java

@@ -118,8 +118,7 @@ public Long deleteParallel(List<String> keys) throws StorageException {
 
     public boolean scheduleAsyncDelete(final String prefix) throws StorageException {
         final long timestamp = currentTimeSeconds();
-        String msg = String.format(
-                "Issuing bulk delete on '%s/%s' for objects older than %d",
+        String msg = "Issuing bulk delete on '%s/%s' for objects older than %d".formatted(
                 client.getContainerName(), prefix, timestamp);
         log.info(msg);
 
@@ -159,8 +158,7 @@ public void issuePendingBulkDeletes() throws StorageException {
                 final String prefix = e.getKey().toString();
                 final long timestamp = Long.parseLong(e.getValue().toString());
                 if (log.isLoggable(Level.INFO))
-                    log.info(String.format(
-                            "Restarting pending bulk delete on '%s/%s':%d",
+                    log.info("Restarting pending bulk delete on '%s/%s':%d".formatted(
                             client.getContainerName(), prefix, timestamp));
                 if (!asyncDelete(prefix, timestamp)) {
                     deletesToClear.add(prefix);
@@ -219,8 +217,7 @@ public Long call() throws Exception {
             try {
                 checkInterrupted();
                 if (log.isLoggable(Level.INFO))
-                    log.info(String.format(
-                            "Running bulk delete on '%s/%s':%d", client.getContainerName(), prefix, timestamp));
+                    log.info("Running bulk delete on '%s/%s':%d".formatted(client.getContainerName(), prefix, timestamp));
 
                 BlobContainerClient container = client.getContainer();
 
@@ -238,23 +235,20 @@ public Long call() throws Exception {
                     }
                 }
             } catch (InterruptedException | IllegalStateException e) {
-                log.info(String.format(
-                        "Azure bulk delete aborted for '%s/%s'. Will resume on next startup.",
+                log.info("Azure bulk delete aborted for '%s/%s'. Will resume on next startup.".formatted(
                         client.getContainerName(), prefix));
                 throw e;
             } catch (RuntimeException e) {
                 log.log(
                         Level.WARNING,
-                        String.format(
-                                "Unknown error performing bulk Azure blobs delete of '%s/%s'",
+                        "Unknown error performing bulk Azure blobs delete of '%s/%s'".formatted(
                                 client.getContainerName(), prefix),
                         e);
                 throw e;
             }
 
             if (log.isLoggable(Level.INFO))
-                log.info(String.format(
-                        "Finished bulk delete on '%s/%s':%d. %d objects deleted",
+                log.info("Finished bulk delete on '%s/%s':%d. %d objects deleted".formatted(
                         client.getContainerName(), prefix, timestamp, count));
 
             clearPendingBulkDelete(prefix, timestamp);
@@ -286,8 +280,7 @@ private void clearPendingBulkDelete(final String prefix, final long timestamp) t
                 if (timestamp >= storedTimestamp) {
                     client.putProperties(pendingDeletesKey, deletes);
                 } else if (log.isLoggable(Level.INFO)) {
-                    log.info(String.format(
-                            "bulk delete finished but there's a newer one ongoing for container '%s/%s'",
+                    log.info("bulk delete finished but there's a newer one ongoing for container '%s/%s'".formatted(
                             client.getContainerName(), prefix));
                 }
             } catch (StorageException e) {
@@ -329,8 +322,7 @@ public Long call() throws Exception {
             try {
                 checkInterrupted();
                 if (log.isLoggable(Level.FINER)) {
-                    log.finer(String.format(
-                            "Running delete delete on list of items on '%s':%s ... (only the first 100 items listed)",
+                    log.finer("Running delete delete on list of items on '%s':%s ... (only the first 100 items listed)".formatted(
                             client.getContainerName(), keys.subList(0, Math.min(keys.size(), 100))));
                 }
 
@@ -348,8 +340,7 @@ public Long call() throws Exception {
             }
 
             if (log.isLoggable(Level.INFO))
-                log.info(String.format(
-                        "Finished bulk delete on %s, %d objects deleted", client.getContainerName(), count));
+                log.info("Finished bulk delete on %s, %d objects deleted".formatted(client.getContainerName(), count));
             return count;
         }
 

geowebcache/azureblob/src/test/java/org/geowebcache/azure/AzureBlobStoreIntegrationTest.java

@@ -539,7 +539,7 @@ private void seed(
             int max = (int) Math.pow(2, z);
             for (int x = 0; x < max; x++) {
                 for (int y = 0; y < max; y++) {
-                    log.fine(String.format("seeding %d,%d,%d", x, y, z));
+                    log.fine("seeding %d,%d,%d".formatted(x, y, z));
                     put(x, y, z, gridset, formatExtension, parameters);
                 }
             }

geowebcache/azureblob/src/test/java/org/geowebcache/testcontainers/azure/AzuriteContainer.java

@@ -13,7 +13,6 @@
  */
 package org.geowebcache.testcontainers.azure;
 
-import static java.lang.String.format;
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assume.assumeTrue;
 
@@ -145,7 +144,7 @@ public int getBlobsPort() {
     }
 
     public String getBlobServiceUrl() {
-        return format("http://localhost:%d/%s", getBlobsPort(), getAccountName());
+        return "http://localhost:%d/%s".formatted(getBlobsPort(), getAccountName());
     }
 
     public AzureBlobStoreData getConfiguration(String container) {

geowebcache/core/pom.xml

@@ -105,6 +105,12 @@
       <groupId>org.springframework</groupId>
       <artifactId>spring-webmvc</artifactId>
     </dependency>
+  <dependency>
+   <groupId>jakarta.annotation</groupId>
+   <artifactId>jakarta.annotation-api</artifactId>
+   <version>1.3.5</version>
+   <scope>provided</scope>
+  </dependency>
     <dependency>
       <groupId>org.springframework</groupId>
       <artifactId>spring-jdbc</artifactId>

geowebcache/core/src/main/java/org/geowebcache/ClientStreamAbortedException.java

@@ -14,6 +14,7 @@
 package org.geowebcache;
 
 import java.io.IOException;
+import java.io.Serial;
 
 /**
  * An IOException that means a {@link ServiceStrategy#getDestination(javax.servlet.http.HttpServletResponse)
@@ -28,6 +29,7 @@
  */
 public final class ClientStreamAbortedException extends IOException {
 
+    @Serial
     private static final long serialVersionUID = -812677957232110980L;
 
     public ClientStreamAbortedException() {