You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
TTL unset from DefaultListFilesCache when CacheManagerConfig::list_files_cache_ttl is not set explicitly #19396
If user creates a ListFilesCache with ttl set, e.g. DefaultListFilesCache::new(1024, Some(Duration::from_secs(1)), but does not set CacheManagerConfig::list_files_cache_ttl, ttl in the list_files_cache would be unset.
To Reproduce
use std::{sync::Arc, time::Duration};
use datafusion_execution::cache::{
DefaultListFilesCache,
cache_manager::{CacheManager, CacheManagerConfig},
};
fn main() {
let list_file_cache = DefaultListFilesCache::new(1024, Some(Duration::from_secs(1)));
let config =
CacheManagerConfig::default().with_list_files_cache(Some(Arc::new(list_file_cache)));
let cache_manager = CacheManager::try_new(&config).unwrap();
// check on is_some() would fail
assert!(
cache_manager
.get_list_files_cache()
.unwrap()
.cache_ttl()
.is_some()
);
}
Activity
shashidhar-bm commentedon Dec 19, 2025
take
fix: preserve ListFilesCache TTL when not set in config (#19401)