Skip to content

Support PostgreSQL numeric type to decimal #806

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Support PostgreSQL numeric type to decimal #806

wants to merge 3 commits into from

Conversation

holicc
Copy link

@holicc holicc commented May 15, 2025

longshan.lu added 3 commits May 15, 2025 19:04
feat(arrow): add support for Decimal128 type in ArrowAssoc and update…
c7d6ee3 
… Postgres transport mappings

- Implemented ArrowAssoc for Decimal and Option<Decimal> with appropriate builders and append methods.
- Added Decimal128 support in the ArrowTypeSystem.
- Updated PostgresArrowTransport to convert Numeric types to Decimal128.
- Enhanced tests for Decimal128 handling in Arrow and Postgres integration.
refactor(postgres): update Decimal conversion in PostgresArrowTransport
918b582 
- Changed the conversion of Numeric types from Float64 to Decimal in Postgres transport mappings.
- Simplified the TypeConversion implementation for Decimal to return Decimal directly.
refactor(postgres): standardize Decimal conversion in PostgresArrowTr…
ca6c7b6 
…ansport

- Updated the conversion mappings for Numeric types to use 'conversion auto' instead of 'conversion option'.
- Removed redundant TypeConversion implementations for Decimal and Vec<Option<Decimal> as they were unnecessary.
# for free to join this conversation on GitHub. Already have an account? # 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
@holicc