Skip to content

fix: partial entry ephemeral cache release - #19872

Merged
clintropolis merged 1 commit into
apache:masterfrom
clintropolis:ephemeral-release-partial-fix
Aug 4, 2026
Merged

fix: partial entry ephemeral cache release#19872
clintropolis merged 1 commit into
apache:masterfrom
clintropolis:ephemeral-release-partial-fix

Conversation

@clintropolis

Copy link
Copy Markdown
Member

Description

Fixes a bug when using partial segment loading in tasks (ephemeral mode) where queries using bundles that depend on other bundles (like projections) leading to a ConcurrentModificationException. Fixed by calling unmount outside of computeIfPresent.

@FrankChen021 FrankChen021 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have reviewed the code for correctness, edge cases, concurrency, and integration risks; no issues found.

Reviewed 3 of 3 changed files.


This is an automated review by Codex GPT-5.6-Sol

@cryptoe cryptoe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM.
Thank you.

@clintropolis
clintropolis merged commit c70397c into apache:master Aug 4, 2026
28 checks passed
@clintropolis
clintropolis deleted the ephemeral-release-partial-fix branch August 4, 2026 20:32
@github-actions github-actions Bot added this to the 39.0.0 milestone Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants