danielcweeks commented on code in PR #16798:
URL: https://github.com/apache/iceberg/pull/16798#discussion_r3508546098
##########
format/spec.md:
##########
@@ -268,7 +268,7 @@ Supported primitive types are defined in the table below.
Primitive types added
| | **`long`** | 64-bit signed integers
|
|
| | **`float`** | [32-bit IEEE
754](https://en.wikipedia.org/wiki/IEEE_754) floating point | Can promote to
double |
| | **`double`** | [64-bit IEEE
754](https://en.wikipedia.org/wiki/IEEE_754) floating point |
|
-| | **`decimal(P,S)`** | Fixed-point decimal; precision P,
scale S | Scale is fixed, precision must be 38
or less |
+| | **`decimal(P, S)`** | Fixed-point decimal; precision P,
scale S | Scale is fixed, precision must be 38
or less |
Review Comment:
This feels unnecessary since it's documentation and it's not about
representation. I wouldn't say this needs to be updated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
