GH-5028 Reduce duplicate use of Unique plan node

hmottestad · hmottestad · commit 87caad7077d0 · 2024-06-12T15:55:20.000+02:00
diff --git a/core/sail/shacl/src/main/java/org/eclipse/rdf4j/sail/shacl/ast/planNodes/Unique.java b/core/sail/shacl/src/main/java/org/eclipse/rdf4j/sail/shacl/ast/planNodes/Unique.java
@@ -61,6 +61,11 @@ public static PlanNode getInstance(PlanNode parent, boolean compress) {
 		if (parent.isGuaranteedEmpty()) {
 			return parent;
 		}
+
+		if (parent instanceof Unique && (!compress || ((Unique) parent).compress == compress)) {
+			return parent;
+		}
+
 		return new Unique(parent, compress);
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -61,6 +61,11 @@ public static PlanNode getInstance(PlanNode parent, boolean compress) {`
`61`	`61`	`if (parent.isGuaranteedEmpty()) {`
`62`	`62`	`return parent;`
`63`	`63`	`}`
	`64`	`+`
	`65`	`+ if (parent instanceof Unique && (!compress \|\| ((Unique) parent).compress == compress)) {`
	`66`	`+ return parent;`
	`67`	`+ }`
	`68`	`+`
`64`	`69`	`return new Unique(parent, compress);`
`65`	`70`	`}`
`66`	`71`