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

Issue #6027: Explicit ROW_GROUP_SIZE no longer breaks IMPORT/EXPORT round-trip #8559

Merged
merged 1 commit into from
Aug 14, 2023

Conversation

jmeulemans
Copy link
Contributor

@jmeulemans jmeulemans commented Aug 13, 2023

fixes #6027

@Tishj
Copy link
Contributor

Tishj commented Aug 13, 2023

The title for your PR is a little confusing, but after looking at the linked issue I get it.

Thanks for fixing this, if you put fixes #6027 in the description it will actually auto-close when this gets merged, now it's just a mention

Maybe a better title would be:
explicit ROW_GROUP_SIZE no longer breaks IMPORT/EXPORT round-trip

@jmeulemans jmeulemans changed the title Issue #6027: Allow ROW_GROUP_SIZE option during parquet read Issue #6027: Explicit ROW_GROUP_SIZE no longer breaks IMPORT/EXPORT round-trip Aug 13, 2023
@jmeulemans
Copy link
Contributor Author

Maybe a better title would be: explicit ROW_GROUP_SIZE no longer breaks IMPORT/EXPORT round-trip

Thanks that's much better

@hannes hannes merged commit d907e40 into duckdb:master Aug 14, 2023
51 checks passed
@hannes
Copy link
Member

hannes commented Aug 14, 2023

Thanks for the fix

@BertTijhuis
Copy link

Thanks for resolving this issue

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.

Export/Import issue regarding parquet and ROW_GROUP_SIZE
4 participants