This is an automated email from the ASF dual-hosted git repository.
terrymanu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git
The following commit(s) were added to refs/heads/master by this push:
new d77c5935b28 Refactor WhereClauseShardingConditionEngine (#39919)
d77c5935b28 is described below
commit d77c5935b2876ab12cb63f438a25a4f6ccf9dbc3
Author: Liang Zhang <[email protected]>
AuthorDate: Tue Sep 22 10:06:43 2026 +0800
Refactor WhereClauseShardingConditionEngine (#39919)
---
.../condition/engine/WhereClauseShardingConditionEngine.java | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git
a/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/route/engine/condition/engine/WhereClauseShardingConditionEngine.java
b/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/route/engine/condition/engine/WhereClauseShardingConditionEngine.java
index 8c439624827..5741a2d9ae0 100644
---
a/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/route/engine/condition/engine/WhereClauseShardingConditionEngine.java
+++
b/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/route/engine/condition/engine/WhereClauseShardingConditionEngine.java
@@ -56,7 +56,6 @@ import java.util.Collection;
import java.util.Collections;
import java.util.HashMap;
import java.util.HashSet;
-import java.util.Iterator;
import java.util.LinkedList;
import java.util.List;
import java.util.Map;
@@ -196,12 +195,7 @@ public final class WhereClauseShardingConditionEngine {
for (Comparable<?> each : value2) {
normalizedValue2.add(normalizeNumber(each));
}
- Iterator<Comparable<?>> iterator = value1.iterator();
- while (iterator.hasNext()) {
- if
(!normalizedValue2.contains(normalizeNumber(iterator.next()))) {
- iterator.remove();
- }
- }
+ value1.removeIf(each ->
!normalizedValue2.contains(normalizeNumber(each)));
return value1;
}
Collection<Comparable<?>> convertedValue2 = value2;