clintropolis opened a new pull request, #19693: URL: https://github.com/apache/druid/pull/19693
### Description After #19683, all partial load operations are now deliberately planned, so this PR begins to clean some stuff up by removing the lazy load behavior of `mapFile`, which prior to this patch would perform a load on demand if the internal file we were trying to map was not present in the segment. `mapFile` now throws if the file is not present since lazy loading in this fashion (like, done inline with processing) is unlikely to ever be desired behavior. Eventually I'd like to consolidate all fetching to be the parallel planned fetches, but some of the refactoring required is a bit more involved than this change (e.g. the full-load fallback the native engines use), so will save this for a follow-up. -- 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]
