Skip to content

Use iterators instead of callbacks in TableIndexList and RowGroupCollection scans#20759

Merged
Mytherin merged 5 commits intoduckdb:v1.5-variegatafrom
Mytherin:betterloops
Jan 31, 2026
Merged

Use iterators instead of callbacks in TableIndexList and RowGroupCollection scans#20759
Mytherin merged 5 commits intoduckdb:v1.5-variegatafrom
Mytherin:betterloops

Conversation

@Mytherin
Copy link
Collaborator

@Mytherin Mytherin commented Jan 31, 2026

So instead of this:

table_info.GetIndexes().Scan([&](Index &index) {
    ...
});

We can do this:

for(auto &index : table_info.GetIndexes().Indexes()) {
}

@Mytherin Mytherin merged commit 6cd1b34 into duckdb:v1.5-variegata Jan 31, 2026
57 checks passed
@carlopi carlopi mentioned this pull request Feb 2, 2026
Merged
Mytherin added a commit that referenced this pull request Feb 2, 2026
@Mytherin
[chore] Skip building spatial, needs a patch, needs to be reverted (#…
77cb0e3 
…20772)

Unsure if we want this in as a temporary measure while conflict between
#20759 and the reintroduction of
`spatial` is sorted out on a side (and then this needs to be reverted)
krlmlr added a commit to krlmlr/duckdb-r that referenced this pull request Feb 28, 2026
@krlmlr
vendor: Update vendored sources to duckdb/duckdb@6cd1b34
b7c6c47 
Date: 2026-01-31 22:32:26 +0100

Use iterators instead of callbacks in TableIndexList and RowGroupCollection scans (duckdb/duckdb#20759)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Feb 28, 2026
@krlmlr
vendor: Update vendored sources to duckdb/duckdb@6cd1b34
8dd86a1 
Date: 2026-01-31 22:32:26 +0100

Use iterators instead of callbacks in TableIndexList and RowGroupCollection scans (duckdb/duckdb#20759)
krlmlr added a commit to krlmlr/duckdb-r that referenced this pull request Feb 28, 2026
@krlmlr
vendor: Update vendored sources to duckdb/duckdb@6cd1b34
dfb65f4 
Date: 2026-01-31 22:32:26 +0100

Use iterators instead of callbacks in TableIndexList and RowGroupCollection scans (duckdb/duckdb#20759)
krlmlr added a commit to krlmlr/duckdb-r that referenced this pull request Feb 28, 2026
@krlmlr
vendor: Update vendored sources to duckdb/duckdb@6cd1b34
f009f38 
Date: 2026-01-31 22:32:26 +0100

Use iterators instead of callbacks in TableIndexList and RowGroupCollection scans (duckdb/duckdb#20759)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Mytherin