Skip to content

[Fix] Reset segment memory when initialising new Prefix#18441

Merged
Mytherin merged 1 commit intoduckdb:v1.3-ossivalisfrom
taniabogatsch:index-fix
Jul 30, 2025
Merged

[Fix] Reset segment memory when initialising new Prefix#18441
Mytherin merged 1 commit intoduckdb:v1.3-ossivalisfrom
taniabogatsch:index-fix

Conversation

@taniabogatsch
Copy link
Contributor

@taniabogatsch taniabogatsch commented Jul 29, 2025

When making changes to indexes, we might reuse previously used index segments. In the case of this bug, we reuse a previous segment in a place where we then expect it to be uninitialised.

I am not sure how to add the reproducer, since the table (with the correct sequence of values to cause this) is 10MB (compressed, all other columns removed).

Fixes #18190

@Mytherin Mytherin merged commit b84467e into duckdb:v1.3-ossivalis Jul 30, 2025
53 of 54 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Jul 30, 2025
[Fix] Reset segment memory when initialising new Prefix (duckdb/duckdb#18441)
[Chore] Tidy test configs (duckdb/duckdb#18426)
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Jul 30, 2025
[Fix] Reset segment memory when initialising new Prefix (duckdb/duckdb#18441)
[Chore] Tidy test configs (duckdb/duckdb#18426)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
@taniabogatsch taniabogatsch deleted the index-fix branch July 30, 2025 09:21
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.

2 participants