huaxingao opened a new pull request, #18057:
URL: https://github.com/apache/iceberg/pull/18057
The UDF spec requires that type strings "contain no spaces or quote
characters", but Iceberg
serializes parameterized type strings with a space after each parameter
separator —
`decimal(9, 2)`, `geography(OGC:CRS84, spherical)`. No implementation that
uses Iceberg's own
type serialization can satisfy that rule, and the spec's own `decimal(9,2)`
example does not
conform to it either.
The spaces can't be normalized away, because a space inside a parameter
value is significant:
`geometry(srid: 3857)` and `geometry(srid:3857)` are different types.
This PR:
- Drops the no-spaces requirement for type strings, and states that type
strings are used and
compared exactly as Iceberg serializes them.
- Fixes the `decimal(9,2)` example to `decimal(9, 2)`.
- Keeps the no-spaces requirement for `definition-id`, but scopes it to the
separators that the
definition-id format itself adds. An embedded type string keeps its own
spaces.
Prerequisite for #15994.
--
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]
