Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix arrow conversion, map doesn't support large offset #10796

Merged
merged 1 commit into from
Feb 23, 2024

Conversation

yiyuanliu
Copy link
Contributor

Fix #10795

@github-actions github-actions bot marked this pull request as draft February 22, 2024 08:09
@Tishj Tishj self-requested a review February 22, 2024 08:25
Copy link
Contributor

@Tishj Tishj left a comment

Choose a reason for hiding this comment

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

Thanks for the changes, looks good to me!
Also, active PRs get marked as draft automatically when new changes are pushed, feel free to just undraft the PR so CI can run 👍

@yiyuanliu yiyuanliu marked this pull request as ready for review February 22, 2024 08:37
@github-actions github-actions bot marked this pull request as draft February 22, 2024 12:34
@yiyuanliu yiyuanliu marked this pull request as ready for review February 22, 2024 12:34
@Mytherin Mytherin merged commit 3c135eb into duckdb:main Feb 23, 2024
45 of 46 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Mar 15, 2024
Merge pull request duckdb/duckdb#10796 from yiyuanliu/lyy/arrow-map
Merge pull request duckdb/duckdb#10692 from maiadegraaf/ODBC_Connect
Merge pull request duckdb/duckdb#10814 from szarnyasg/update-readme
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.

Bug in map to Arrow conversion.
3 participants