Address #1721 by putting filteredPKStr used for paging in QueryIterator first in the query, allowing any RANDOM_SAMPLE clause to come last (#1722) (#1725)

yhmo · gehnaphore · George Hoffman · web-flow · commit a2cb7bec546b · 2025-12-17T13:05:53.000+08:00
Co-authored-by: gehnaphore &lt;geh@georgeandjulia.com&gt;
Co-authored-by: George Hoffman &lt;george@patientcase.com&gt;
diff --git a/sdk-core/src/main/java/io/milvus/orm/iterator/QueryIterator.java b/sdk-core/src/main/java/io/milvus/orm/iterator/QueryIterator.java
@@ -199,7 +199,7 @@ private String setupNextExpr() {
         if (StringUtils.isEmpty(currentExpr)) {
             return filteredPKStr;
         }
-        return " ( " + currentExpr + " ) " + " and " + filteredPKStr;
+        return filteredPKStr + " and ( " + currentExpr + " )";
     }
 
     private boolean isResSufficient(List<QueryResultsWrapper.RowRecord> ret) {

Original file line number	Diff line number	Diff line change
`@@ -199,7 +199,7 @@ private String setupNextExpr() {`
`199`	`199`	`if (StringUtils.isEmpty(currentExpr)) {`
`200`	`200`	`return filteredPKStr;`
`201`	`201`	`}`
`202`		`- return " ( " + currentExpr + " ) " + " and " + filteredPKStr;`
	`202`	`+ return filteredPKStr + " and ( " + currentExpr + " )";`
`203`	`203`	`}`
`204`	`204`
`205`	`205`	`private boolean isResSufficient(List<QueryResultsWrapper.RowRecord> ret) {`