You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ja...@apache.org on 2026/01/30 07:40:02 UTC

(lucene) branch main updated: Minor refactor in SegmentTermsEnum.java (#15624)

This is an automated email from the ASF dual-hosted git repository.

jainankitk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/lucene.git


The following commit(s) were added to refs/heads/main by this push:
     new cdeb4da1327 Minor refactor in SegmentTermsEnum.java  (#15624)
cdeb4da1327 is described below

commit cdeb4da132714f815ee0f395bd1ccd15b564790f
Author: Shubham Sharma <sh...@gmail.com>
AuthorDate: Fri Jan 30 07:39:56 2026 +0000

    Minor refactor in SegmentTermsEnum.java  (#15624)
---
 .../lucene103/blocktree/SegmentTermsEnum.java      | 27 +++++-----------------
 1 file changed, 6 insertions(+), 21 deletions(-)

diff --git a/lucene/core/src/java/org/apache/lucene/codecs/lucene103/blocktree/SegmentTermsEnum.java b/lucene/core/src/java/org/apache/lucene/codecs/lucene103/blocktree/SegmentTermsEnum.java
index 99acaae560d..c0c0991dfce 100644
--- a/lucene/core/src/java/org/apache/lucene/codecs/lucene103/blocktree/SegmentTermsEnum.java
+++ b/lucene/core/src/java/org/apache/lucene/codecs/lucene103/blocktree/SegmentTermsEnum.java
@@ -434,27 +434,7 @@ final class SegmentTermsEnum extends BaseTermsEnum {
           return null;
         }
 
-        if (prefetch) {
-          currentFrame.prefetchBlock();
-        }
-
-        return () -> {
-          currentFrame.loadBlock();
-
-          final SeekStatus result = currentFrame.scanToTerm(target, true);
-          if (result == SeekStatus.FOUND) {
-            // if (DEBUG) {
-            //   System.out.println("  return FOUND term=" + term.utf8ToString() + " " + term);
-            // }
-            return true;
-          } else {
-            // if (DEBUG) {
-            //   System.out.println("  got " + result + "; return NOT_FOUND term=" +
-            // ToStringUtils.bytesRefToString(term));
-            // }
-            return false;
-          }
-        };
+        return getIOBooleanSupplier(target, prefetch);
       } else {
         // Follow this node
         node = nextNode;
@@ -491,6 +471,11 @@ final class SegmentTermsEnum extends BaseTermsEnum {
       return null;
     }
 
+    return getIOBooleanSupplier(target, prefetch);
+  }
+
+  private IOBooleanSupplier getIOBooleanSupplier(BytesRef target, boolean prefetch)
+      throws IOException {
     if (prefetch) {
       currentFrame.prefetchBlock();
     }