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

fix(postgres): Handling of the decimal column in the build query #17

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

Toru-Takagi
Copy link
Contributor

@Toru-Takagi Toru-Takagi commented Jan 10, 2024

fiix: #18

Problem

When creating a decimal(19, 4) column in PostgreSQL, it generates a data class with a property of type java.math.BigDecimal? .

However, when generating a query for a table with this column, the code results.getjava.math.BigDecimal() is produced. It seems that the correct code should be results.getBigDecimal() .

@Toru-Takagi Toru-Takagi changed the title Handling of the decimal column in the build query fix(postgres): Handling of the decimal column in the build query Jan 13, 2024
@kyleconroy
Copy link
Contributor

Thanks for the fix! I'll cut a new release in a few days.

@kyleconroy kyleconroy merged commit d3900db into sqlc-dev:main Jan 22, 2024
2 checks passed
@Toru-Takagi
Copy link
Contributor Author

Thank you for checking despite your busy schedule.

@Toru-Takagi
Copy link
Contributor Author

@kyleconroy

I apologize for the inconvenience while you are busy with the Go and TypeScript support. Will it take much longer for the release?

It's tough having to manually rewrite every time I generate. Other team members are also having trouble.

I'm sorry for the inconvenience, but I appreciate your consideration. 🙇

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.

2 participants