[VL] Enable file handle cache by default with TTL-based eviction#12400

Open

iemejia wants to merge 25 commits into

apache:mainfrom

iemejia:feature/velox-enable-file-handle-cache-default

Commits on Jul 10, 2026

[VL] Enable file handle cache by default with TTL-based eviction
iemejia
committed
Address review comments: fix schema assertion, deletion test, patch guard, cache default
iemejia
committed
Tighten fileCount assertion to >= 200 to match repartition(200)
iemejia
committed
Add TTL eviction test: verify scans succeed after cached handles expire
iemejia
committed
Narrow catch to file-not-found errors; use count() instead of collect().length
iemejia
committed
Regenerate configuration docs for new file handle cache configs
iemejia
committed
Add value validation for numCacheFileHandles and fileHandleExpirationDurationMs
iemejia
committed
Clarify that remote FS cache entries still count toward OS limits
iemejia
committed
Add checkGlutenPlan assertions to all file handle cache tests
iemejia
committed
Fix config doc padding and address review comments
iemejia
committed
Remove trailing whitespace; reword cache-hit comments
iemejia
committed
Fix spark-submit example in FileHandleCacheBenchmark scaladoc
iemejia
committed
Add file handle cache configs to native conf defaults list
iemejia
committed
Use standard a/b prefixes in file-handle-cache-ttl.patch
iemejia
committed
Rename repeated-scan test to drop 'cache hit path' claim
iemejia
committed
Use timeConf for fileHandleExpirationDurationMs
iemejia
committed
Extract TTL constant; remove remaining 'cache hit path' comments
iemejia
committed
Add checkValue validation for ssdCacheIOThreads
iemejia
committed
Revert timeConf back to longConf for fileHandleExpirationDurationMs
iemejia
committed
Use timeConf for fileHandleExpirationDurationMs
iemejia
committed
Quote path expansions in apply_compilation_fixes for safety
iemejia
committed
Make apply_compilation_fixes self-contained; clarify TTL test scope
iemejia
committed
Harden deletion test: check cause chain before message matching
iemejia
committed
Rename TTL test to reflect what it actually asserts
iemejia
committed
Fix scalafmt: move case guard to its own line
iemejia
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[VL] Enable file handle cache by default with TTL-based eviction#12400

[VL] Enable file handle cache by default with TTL-based eviction#12400
iemejia wants to merge 25 commits into
apache:mainfrom
iemejia:feature/velox-enable-file-handle-cache-default

Commits on Jul 10, 2026

[VL] Enable file handle cache by default with TTL-based eviction

Address review comments: fix schema assertion, deletion test, patch guard, cache default

Tighten fileCount assertion to >= 200 to match repartition(200)

Add TTL eviction test: verify scans succeed after cached handles expire

Narrow catch to file-not-found errors; use count() instead of collect().length

Regenerate configuration docs for new file handle cache configs

Add value validation for numCacheFileHandles and fileHandleExpirationDurationMs

Clarify that remote FS cache entries still count toward OS limits

Add checkGlutenPlan assertions to all file handle cache tests

Fix config doc padding and address review comments

Remove trailing whitespace; reword cache-hit comments

Fix spark-submit example in FileHandleCacheBenchmark scaladoc

Add file handle cache configs to native conf defaults list

Use standard a/b prefixes in file-handle-cache-ttl.patch

Rename repeated-scan test to drop 'cache hit path' claim

Use timeConf for fileHandleExpirationDurationMs

Extract TTL constant; remove remaining 'cache hit path' comments

Add checkValue validation for ssdCacheIOThreads

Revert timeConf back to longConf for fileHandleExpirationDurationMs

Use timeConf for fileHandleExpirationDurationMs

Quote path expansions in apply_compilation_fixes for safety

Make apply_compilation_fixes self-contained; clarify TTL test scope

Harden deletion test: check cause chain before message matching

Rename TTL test to reflect what it actually asserts

Fix scalafmt: move case guard to its own line