Re: [I] Make the CompletionTimeQueryView loading incremental [hudi]

["nsivabalan (via GitHub)" <gi...@apache.org>]

nsivabalan commented on issue #16255:
URL: https://github.com/apache/hudi/issues/16255#issuecomment-4795564366

   This appears to already be addressed on master. `CompletionTimeQueryViewV2` no longer eagerly loads the archived timeline:
   
   - The constructor's `load()` only reads completed instants from the **active** timeline (`hudi-common/src/main/java/org/apache/hudi/common/table/timeline/versioning/v2/CompletionTimeQueryViewV2.java`, ~L296).
   - Archived instants are loaded on demand via `loadCompletionTimeIncrementally(startTime)` (~L278), invoked from `getCompletionTime()` and `getInstantTimes()` when a query reaches below the current `cursorInstant`. The cursor slides as older queries arrive, so subsequent queries reuse the cache.
   
   Incremental loading was introduced in HUDI-6929 (commit `bb8fc3e9f6`) and refined further in #14013, #14019, and #17914 (the last of which removes the remaining eager-load path and broadens caching). If this issue was tracking something more specific that's still missing, please reopen with details; otherwise this can be closed as completed.


-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org