Skip to content

Conversation

@aykut-bozkurt
Copy link
Member

Arrow supports extension types with new major version 54. They can correctly convert uuid and json arrow arrays from/to parquet. This PR uses this fact to natively read/write these types from/to parquet to/from postgres.

Closes #46.

@aykut-bozkurt aykut-bozkurt requested a review from marcoslot March 6, 2025 19:02
Base automatically changed from aykut/update-deps to main March 7, 2025 08:44
Arrow supports extension types with new major version 54. They can correctly convert
uuid and json arrow arrays from/to parquet. This PR uses this fact to natively read/write
these types from/to parquet to/from postgres.

Closes #46.
@aykut-bozkurt aykut-bozkurt force-pushed the aykut/extension-types branch from 457c806 to 854b607 Compare March 7, 2025 08:52
@codecov
Copy link

codecov bot commented Mar 7, 2025

Codecov Report

Attention: Patch coverage is 96.32107% with 11 lines in your changes missing coverage. Please review.

Project coverage is 91.84%. Comparing base (6014e11) to head (854b607).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/arrow_parquet/schema_parser.rs 72.50% 11 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #114      +/-   ##
==========================================
+ Coverage   91.72%   91.84%   +0.12%     
==========================================
  Files          77       83       +6     
  Lines       10232    10494     +262     
==========================================
+ Hits         9385     9638     +253     
- Misses        847      856       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aykut-bozkurt aykut-bozkurt merged commit 5766819 into main Mar 11, 2025
6 checks passed
@aykut-bozkurt aykut-bozkurt deleted the aykut/extension-types branch March 11, 2025 09:12
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.

Support UUID type

3 participants