binarybrain-009 opened a new pull request, #18589: URL: https://github.com/apache/iceberg/pull/18589
The analytics accelerator stream factory cache is bounded at 100 entries and its removal listener closes evicted factories, so once a JVM has more than 100 live `S3FileIO` instances, reads through an evicted factory hang. Every entry already has an owner that removes it: closing the `S3FileIO` calls `cleanupCache`. This drops the size bound so a factory is closed only when its `S3FileIO` is closed. FileIOs that are never closed keep their factory cached, but they already keep their S3 clients open. Closes #17485 Test plan: `TestAnalyticsAcceleratorUtil` opens 150 FileIOs and checks that every factory is still cached after cache maintenance. It fails with the size bound and passes without it. --- **AI Disclosure** - Model: Claude Opus 5.5 - Platform/Tool: Claude Code - Human Oversight: fully reviewed - Prompt Summary: Fix #17485 so the analytics accelerator factory cache no longer closes factories of live S3FileIO instances; the fix and its test are generated. -- 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]
