zhangstar333 commented on code in PR #67650:
URL: https://github.com/apache/doris/pull/67650#discussion_r3975238239


##########
fe/fe-core/src/main/java/org/apache/doris/datasource/lance/LanceMetadataLoader.java:
##########
@@ -64,17 +66,10 @@ public static LanceTableMetadata loadLatestForTvf(
      * <p>Called by
      * {@link LanceExternalCatalog#loadTableMetadata(String, String, 
java.util.Optional)} when no
      * time-travel version is requested. Schema, version, and fragments are 
read from the same
-     * opened dataset snapshot.
+     * opened dataset snapshot, together with index coverage for fragment 
grouping and external searches.
      */
     public static LanceTableMetadata loadLatest(String datasetUri,
             Map<String, String> lanceStorageOptions, BufferAllocator 
allocator) throws Exception {
-        return loadInternal(
-                datasetUri, lanceStorageOptions, OptionalLong.empty(), 
allocator, false);
-    }
-
-    /** Loads the latest fixed snapshot together with search-index segment 
coverage. */
-    public static LanceTableMetadata loadLatestWithIndexSegments(
-            String datasetUri, Map<String, String> lanceStorageOptions, 
BufferAllocator allocator) throws Exception {
         return loadInternal(
                 datasetUri, lanceStorageOptions, OptionalLong.empty(), 
allocator, true);

Review Comment:
   need in another pr fixed the index load order and failed process



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to