Enable compilation failures on warnings for driver module main sources (#1275)

This is primarily an extra protection from unintended warnings.

Author: injectives

driver/clirr-ignored-differences.xml

@@ -334,4 +334,10 @@
         <method>java.lang.String neo4jErrorCode()</method>
     </difference>
 
+    <difference>
+        <className>org/neo4j/driver/QueryRunner</className>
+        <differenceType>7012</differenceType>
+        <method>void close()</method>
+    </difference>
+
 </differences>

driver/src/main/java/org/neo4j/driver/Config.java

@@ -22,6 +22,7 @@
 import static org.neo4j.driver.internal.logging.DevNullLogging.DEV_NULL_LOGGING;
 
 import java.io.File;
+import java.io.Serial;
 import java.io.Serializable;
 import java.net.InetAddress;
 import java.util.ArrayList;
@@ -70,6 +71,7 @@
  */
 @Immutable
 public class Config implements Serializable {
+    @Serial
     private static final long serialVersionUID = -4496545746399601108L;
 
     private static final Config EMPTY = builder().build();

driver/src/main/java/org/neo4j/driver/QueryRunner.java

@@ -23,4 +23,7 @@
  *
  * @since 1.0
  */
-public interface QueryRunner extends SimpleQueryRunner, AutoCloseable {}
+public interface QueryRunner extends SimpleQueryRunner, AutoCloseable {
+    @Override
+    void close() throws RuntimeException;
+}

driver/src/main/java/org/neo4j/driver/SessionConfig.java

@@ -21,6 +21,7 @@
 import static java.util.Objects.requireNonNull;
 import static org.neo4j.driver.internal.handlers.pulln.FetchSizeUtil.assertValidFetchSize;
 
+import java.io.Serial;
 import java.io.Serializable;
 import java.util.Arrays;
 import java.util.Objects;
@@ -34,6 +35,7 @@
  * The session configurations used to configure a session.
  */
 public class SessionConfig implements Serializable {
+    @Serial
     private static final long serialVersionUID = 5773462156979050657L;
 
     private static final SessionConfig EMPTY = builder().build();

driver/src/main/java/org/neo4j/driver/TransactionConfig.java

@@ -23,6 +23,7 @@
 import static java.util.Objects.requireNonNull;
 import static org.neo4j.driver.internal.util.Preconditions.checkArgument;
 
+import java.io.Serial;
 import java.io.Serializable;
 import java.time.Duration;
 import java.util.HashMap;
@@ -63,6 +64,7 @@
  * @see Session
  */
 public class TransactionConfig implements Serializable {
+    @Serial
     private static final long serialVersionUID = -7954949878657177280L;
 
     private static final TransactionConfig EMPTY = builder().build();

driver/src/main/java/org/neo4j/driver/Values.java

@@ -570,12 +570,23 @@ public static Function<Value, Entity> ofEntity() {
     /**
      * Converts values to {@link Long entity id}.
      *
+     * @deprecated superseded by {@link #ofEntityElementId()}.
      * @return a function that returns the id an entity {@link Value}
      */
+    @Deprecated
     public static Function<Value, Long> ofEntityId() {
         return val -> val.asEntity().id();
     }
 
+    /**
+     * Converts values to {@link String element id}.
+     *
+     * @return a function that returns the element id of an entity {@link Value}
+     */
+    public static Function<Value, String> ofEntityElementId() {
+        return val -> val.asEntity().elementId();
+    }
+
     /**
      * Converts values to {@link Node}.
      *

driver/src/main/java/org/neo4j/driver/exceptions/AuthenticationException.java

@@ -18,6 +18,8 @@
  */
 package org.neo4j.driver.exceptions;
 
+import java.io.Serial;
+
 /**
  * Failed to authenticate the driver to the server due to bad credentials provided.
  * When this error happens, the error could be recovered by closing the current driver and restart a new driver with
@@ -26,6 +28,9 @@
  * @since 1.1
  */
 public class AuthenticationException extends SecurityException {
+    @Serial
+    private static final long serialVersionUID = 1324352999966240271L;
+
     public AuthenticationException(String code, String message) {
         super(code, message);
     }

driver/src/main/java/org/neo4j/driver/exceptions/AuthorizationExpiredException.java

@@ -18,12 +18,17 @@
  */
 package org.neo4j.driver.exceptions;
 
+import java.io.Serial;
+
 /**
  * The authorization info maintained on the server has expired. The client should reconnect.
  * <p>
  * Error code: Neo.ClientError.Security.AuthorizationExpired
  */
 public class AuthorizationExpiredException extends SecurityException implements RetryableException {
+    @Serial
+    private static final long serialVersionUID = 5688002170978405558L;
+
     public static final String DESCRIPTION =
             "Authorization information kept on the server has expired, this connection is no longer valid.";
 

driver/src/main/java/org/neo4j/driver/exceptions/ClientException.java

@@ -18,12 +18,17 @@
  */
 package org.neo4j.driver.exceptions;
 
+import java.io.Serial;
+
 /**
  * A <em>ClientException</em> indicates that the client has carried out an operation incorrectly.
  * The error code provided can be used to determine further detail for the problem.
  * @since 1.0
  */
 public class ClientException extends Neo4jException {
+    @Serial
+    private static final long serialVersionUID = -6732913155228185887L;
+
     public ClientException(String message) {
         super(message);
     }

driver/src/main/java/org/neo4j/driver/exceptions/ConnectionReadTimeoutException.java

@@ -18,12 +18,17 @@
  */
 package org.neo4j.driver.exceptions;
 
+import java.io.Serial;
+
 /**
  * Indicates that read timed out due to it taking longer than the server-supplied timeout value via the {@code connection.recv_timeout_seconds} configuration
  * hint. The server might provide this value to clients to let them know when a given connection may be considered broken if client does not get any
  * communication from the server within the specified timeout period. This results in the server being removed from the routing table.
  */
 public class ConnectionReadTimeoutException extends ServiceUnavailableException {
+    @Serial
+    private static final long serialVersionUID = -9222586212813330140L;
+
     public static final ConnectionReadTimeoutException INSTANCE = new ConnectionReadTimeoutException(
             "Connection read timed out due to it taking longer than the server-supplied timeout value via configuration hint.");