Skip to content
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

fix(target-s3-parquet): add a function to conver Decimal values #39

Merged
merged 1 commit into from
Jan 31, 2024
Merged

fix(target-s3-parquet): add a function to conver Decimal values #39

merged 1 commit into from
Jan 31, 2024

Conversation

Marcos314
Copy link
Contributor

@Marcos314 Marcos314 commented Jan 31, 2024 •
edited
Loading

This PR fixes the issue with serializing Decimal objects in JSON. We created a new function within the json.dumps method to deal with Decimal objects.

Changes:

  • Defined a new function decimal_to_str that converts Decimal objects to strings if they are found during JSON serialization.

@Marcos314 Marcos314 requested a review from a team as a code owner January 31, 2024 12:03
@Marcos314 Marcos314 merged commit 94b1b00 into main Jan 31, 2024
3 checks passed
@Marcos314 Marcos314 deleted the PDD-995-2 branch February 7, 2024 18:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@lorransr lorransr lorransr approved these changes

@omaucosta omaucosta omaucosta approved these changes

@stailer37 stailer37 Awaiting requested review from stailer37 stailer37 is a code owner automatically assigned from gupy-io/plataforma-de-dados

@gilsoncostag gilsoncostag Awaiting requested review from gilsoncostag gilsoncostag is a code owner automatically assigned from gupy-io/plataforma-de-dados

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Marcos314 @lorransr @omaucosta