GH-5744: Migrate rdf4j-query to Jackson 3.1.x

- Removed JsonProcessingException import (now JacksonException,
unchecked — no longer needed)
- Added JsonMapper import (tools.jackson.databind.json.JsonMapper)
- Replaced mutable new ObjectMapper() + chained setters with an
immutable JsonMapper.builder() configured via changeDefaultVisibility()
and changeDefaultPropertyInclusion()
- Removed try/catch in toJson() since Jackson 3 exceptions are unchecked

Author: aschwarte10

core/query/pom.xml

@@ -25,12 +25,14 @@
 			<artifactId>jackson-annotations</artifactId>
 		</dependency>
 		<dependency>
-			<groupId>com.fasterxml.jackson.core</groupId>
+			<groupId>tools.jackson.core</groupId>
 			<artifactId>jackson-core</artifactId>
+			<version>${jackson3.version}</version>
 		</dependency>
 		<dependency>
-			<groupId>com.fasterxml.jackson.core</groupId>
+			<groupId>tools.jackson.core</groupId>
 			<artifactId>jackson-databind</artifactId>
+			<version>${jackson3.version}</version>
 		</dependency>
 		<dependency>
 			<groupId>org.apache.commons</groupId>

core/query/src/main/java/org/eclipse/rdf4j/query/explanation/ExplanationImpl.java

@@ -15,8 +15,9 @@
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.PropertyAccessor;
-import com.fasterxml.jackson.core.JsonProcessingException;
-import com.fasterxml.jackson.databind.ObjectMapper;
+
+import tools.jackson.databind.ObjectMapper;
+import tools.jackson.databind.json.JsonMapper;
 
 /**
  * This is an experimental feature. It may be changed, moved or potentially removed in a future release.
@@ -37,7 +38,12 @@ public ExplanationImpl(GenericPlanNode genericPlanNode, boolean timedOut, Object
 		}
 	}
 
-	ObjectMapper objectMapper = new ObjectMapper();
+	ObjectMapper objectMapper = JsonMapper.builder()
+			.changeDefaultVisibility(vc -> vc
+					.withVisibility(PropertyAccessor.ALL, JsonAutoDetect.Visibility.PUBLIC_ONLY)
+					.withVisibility(PropertyAccessor.FIELD, JsonAutoDetect.Visibility.NONE))
+			.changeDefaultPropertyInclusion(incl -> incl.withValueInclusion(JsonInclude.Include.NON_NULL))
+			.build();
 
 	@Override
 	public Object tupleExpr() {
@@ -51,16 +57,10 @@ public GenericPlanNode toGenericPlanNode() {
 
 	@Override
 	public String toJson() {
-		try {
-			// TODO: Consider removing pretty printer
-			return this.objectMapper.setVisibility(PropertyAccessor.ALL, JsonAutoDetect.Visibility.PUBLIC_ONLY)
-					.setVisibility(PropertyAccessor.FIELD, JsonAutoDetect.Visibility.NONE)
-					.setSerializationInclusion(JsonInclude.Include.NON_NULL)
-					.writerWithDefaultPrettyPrinter()
-					.writeValueAsString(toGenericPlanNode());
-		} catch (JsonProcessingException e) {
-			throw new RuntimeException(e);
-		}
+		// TODO: Consider removing pretty printer
+		return this.objectMapper
+				.writerWithDefaultPrettyPrinter()
+				.writeValueAsString(toGenericPlanNode());
 	}
 
 	@Override

pom.xml

@@ -63,6 +63,7 @@
 		<httpcore5.version>5.3.4</httpcore5.version>
 		<jackson.version>2.21.0</jackson.version>
 		<jackson.annotations.version>2.21</jackson.annotations.version>
+		<jackson3.version>3.1.2</jackson3.version>
 		<httpcore.version>4.4.16</httpcore.version>
 		<jsonldjava.version>0.13.4</jsonldjava.version>
 		<last.japicmp.compare.version>6.0.0</last.japicmp.compare.version>