Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update: Add "Float" and "Real" data type support for Postgres BinaryArithmeticOperation #1193

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

codebreaker32
Copy link

@codebreaker32 codebreaker32 commented Mar 29, 2025

Hi @mrigger @robins
Previously, Only Integer data type is supported as discussed in issue #630

I have added the support for

  • Real data type
  • Float data type

Resolves #630
Suggestions to improve are welcome!

@codebreaker32 codebreaker32 changed the title Update: Add "Float" and "Real" data type support for Postgres BinaryArithmeticOperation Update: Add "Float" and "Real" data type support for Postgres BinaryArithmeticOperation. Resolves #630 Mar 29, 2025
@codebreaker32 codebreaker32 changed the title Update: Add "Float" and "Real" data type support for Postgres BinaryArithmeticOperation. Resolves #630 Update: Add "Float" and "Real" data type support for Postgres BinaryArithmeticOperation Mar 29, 2025
@codebreaker32
Copy link
Author

From discussions of issue #630, What I could infer is that we need to add support for more datatypes. This is my attempt to add support for REAL and FLOAT

Please review it @mrigger

@robins
Copy link
Collaborator

robins commented Apr 7, 2025

@codebreaker32

Could you point me to how you ensured that the float numbers are exact?

@codebreaker32
Copy link
Author

codebreaker32 commented Apr 8, 2025

@robins
Sorry I missed that. I used createDouble functions that were already defined. Should I use BigDecimal now?

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.

Possible bugs in postgres ast
2 participants