GH-5775 Avoid regenerating ArrayBindingSet mask

hmottestad · hmottestad · commit 44d3d92963ff · 2026-04-22T17:50:43.000+02:00
diff --git a/core/queryalgebra/evaluation/src/main/java/org/eclipse/rdf4j/query/algebra/evaluation/ArrayBindingSet.java b/core/queryalgebra/evaluation/src/main/java/org/eclipse/rdf4j/query/algebra/evaluation/ArrayBindingSet.java
@@ -144,7 +144,8 @@ public ArrayBindingSet(ArrayBindingSet toCopy, String[] names,
 		this.empty = toCopy.empty;
 		this.cachedSize = toCopy.cachedSize;
 		if (sharedSortedBindingNamesCache != null) {
-			this.activeBindingMask = calculateActiveBindingMask();
+			this.activeBindingMask = toCopy.bindingNames == names ? toCopy.activeBindingMask
+					: calculateActiveBindingMask();
 		}
 		assert !this.empty || size() == 0;
 	}

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,8 @@ public ArrayBindingSet(ArrayBindingSet toCopy, String[] names,`
`144`	`144`	`this.empty = toCopy.empty;`
`145`	`145`	`this.cachedSize = toCopy.cachedSize;`
`146`	`146`	`if (sharedSortedBindingNamesCache != null) {`
`147`		`- this.activeBindingMask = calculateActiveBindingMask();`
	`147`	`+ this.activeBindingMask = toCopy.bindingNames == names ? toCopy.activeBindingMask`
	`148`	`+ : calculateActiveBindingMask();`
`148`	`149`	`}`
`149`	`150`	`assert !this.empty \|\| size() == 0;`
`150`	`151`	`}`