-
Notifications
You must be signed in to change notification settings - Fork 30
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
Remove FLOAT SQL datatype #980
Conversation
Fixes #950 |
This needs some code removal in the web-console too @Karakatiza666 |
CHANGELOG.md
Outdated
@@ -10,6 +10,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 | |||
## [Unreleased] | |||
|
|||
### Added | |||
|
|||
- Removed suppoer for FLOAT SQL data type, since it is ambiguous. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Removed suppoer for FLOAT SQL data type, since it is ambiguous. | |
### Removed | |
- Removed support for FLOAT SQL data type, since it is ambiguous. | |
### Added |
@@ -17,7 +17,7 @@ The compiler supports the following SQL data types: | |||
digits) and a scale (number of decimal digits after period). For example, | |||
23.456 has a precision of 5 and a scale of 3. If scale is missing it is | |||
assumed to be 0. | |||
- `FLOAT`, or `FLOAT4`, or `FLOAT32`, or `REAL`, an IEEE 32-bit floating point number | |||
- `REAL`, or `FLOAT4`, or `FLOAT32`, an IEEE 32-bit floating point number |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are FLOAT4 and FLOAT32 accepted types?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
Is this a user-visible change (yes/no): yes