akashmalbari opened a new pull request, #16612:
URL: https://github.com/apache/iceberg/pull/16612
Fixes #16596.
This updates REST/OpenAPI schema field default handling so `initial-default`
and `write-default` can represent valid Iceberg struct defaults such as `{}`.
The change keeps expression literal values restricted to primitive values
while allowing schema field defaults to use the broader default-value
representation required by the Iceberg v3 spec.
Added regression coverage for:
- struct field defaults represented as `{}`
- nested field defaults inside that struct
- round-tripping schema/default JSON
- REST create-table/schema parsing with struct defaults
Validation:
-
JAVA_HOME=/opt/homebrew/Cellar/openjdk@17/17.0.19/libexec/openjdk.jdk/Contents/Home
./gradlew :iceberg-core:test --tests org.apache.iceberg.TestSchemaParser
--tests org.apache.iceberg.rest.requests.TestCreateTableRequest --stacktrace
-
JAVA_HOME=/opt/homebrew/Cellar/openjdk@17/17.0.19/libexec/openjdk.jdk/Contents/Home
./gradlew :iceberg-open-api:check --stacktrace
-
JAVA_HOME=/opt/homebrew/Cellar/openjdk@17/17.0.19/libexec/openjdk.jdk/Contents/Home
./gradlew spotlessApply
-
JAVA_HOME=/opt/homebrew/Cellar/openjdk@17/17.0.19/libexec/openjdk.jdk/Contents/Home
./gradlew spotlessCheck
--
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]
