Skip to content

[server] include time index in remote log cache weight#3733

Open
zuston wants to merge 1 commit into
apache:mainfrom
zuston:fixremotecache
Open

[server] include time index in remote log cache weight#3733
zuston wants to merge 1 commit into
apache:mainfrom
zuston:fixremotecache

Conversation

@zuston

@zuston zuston commented Jul 22, 2026

Copy link
Copy Markdown
Member

Purpose

Linked issue: close #3732

The time index size is not included in the cache weight. As a result, the cache underestimates its actual storage usage and may retain more index files than allowed by remote.log.index.file.cache.size .

Brief change log

  • Include both offset index and time index sizes in the remote log cache weight.
  • Prevent integer overflow when converting the cache weight for Caffeine.
  • Add a regression test to verify the corrected weight calculation.

Tests

API and Format

Documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect remote cache capacity calculation that miss timeIndex size

1 participant