BlakeOrth commented on PR #19366: URL: https://github.com/apache/datafusion/pull/19366#issuecomment-3671555542
@alamb > I can't set a TTL to 0 This was an intentional decision implemented in https://github.com/apache/datafusion/pull/19108 because setting a TTL of 0 conceptually doesn't make much sense to me because it would mean that all cache entries would immediately expire. Functionally it would allow the cache to consume memory and never hit, which also seems undesirable. That being said, the fact that it caused confusion probably means we can improve the user experience in some aspects. What would you expect a TTL of 0 to do? > Even when I set a ttl, the cache didn't reset This does seem odd. I think what you have shown here should be working as you expect and I'm concerned there might be a bug with timing/expiration that's working in unit tests but not in regular usage. I will devote some time to looking into this! -- 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]
