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.

Sign up for GitHub

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

Jump to bottom

Casting a Float column that contains a NaN to Decimal throws a Panic. #11867

Open
radeusgd opened this issue Dec 13, 2024 · 1 comment
Open

Casting a Float column that contains a NaN to Decimal throws a Panic. #11867

radeusgd opened this issue Dec 13, 2024 · 1 comment
Assignees
@GregoryTravis
Labels
--bug Type: bug -libs Libraries: New libraries to be implemented l-table-column-datatypes

Comments

@radeusgd
Copy link
Member

As can be seen on the screenshot from today's nightly:
image

Apparently we forgot to handle/test this edge case.
@radeusgd radeusgd added --bug Type: bug -libs Libraries: New libraries to be implemented labels Dec 13, 2024
@radeusgd
Copy link
Member Author

Expected behaviour - like with cast to Integer, it should replace it with Nothing and report a warning:
image

@GregoryTravis GregoryTravis self-assigned this Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GregoryTravis GregoryTravis

Labels
--bug Type: bug -libs Libraries: New libraries to be implemented l-table-column-datatypes
Projects
Issues Board
Status: ❓New
Milestone
No milestone
Development

No branches or pull requests
2 participants
@radeusgd @GregoryTravis