Skip to content

Conversation

taniabogatsch
Copy link
Contributor

Fixes #15924.
Fixes #14909.

@Mytherin
Copy link
Collaborator

Thanks for the PR! Looks good - could you just look at the CI failures?

@taniabogatsch
Copy link
Contributor Author

Added the label so we can remove the limitation-docs on this once merged.

@duckdb-draftbot duckdb-draftbot marked this pull request as draft January 29, 2025 10:20
@taniabogatsch taniabogatsch marked this pull request as ready for review January 29, 2025 11:25
@duckdb-draftbot duckdb-draftbot marked this pull request as draft January 29, 2025 11:29
@taniabogatsch taniabogatsch marked this pull request as ready for review January 29, 2025 11:29
@Mytherin
Copy link
Collaborator

Thanks! LGTM

@Mytherin Mytherin merged commit 9e2956a into duckdb:v1.2-histrionicus Jan 29, 2025
51 checks passed
@taniabogatsch taniabogatsch deleted the copy-index branch January 30, 2025 10:34
Mytherin added a commit that referenced this pull request Jan 31, 2025
#15964

I was hoping we could copy the index without having to bind it, but that
causes some issues. So, this PR addresses them by binding the index.

Fix duckdblabs/duckdb-internal#4105
Fix duckdblabs/duckdb-internal#4104
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Documentation Use for issues or PRs that require changes in the documentation Ready To Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants