yuqi1129 opened a new pull request, #10382:
URL: https://github.com/apache/gravitino/pull/10382
### What changes were proposed in this pull request?
This PR fixes so users do not need to explicitly pass .
Changes:
- In , changed to primitive with default value .
- Kept constructor compatibility by accepting , normalizing to .
- Added a regression test in to verify deserializing without works and
produces .
### Why are the changes needed?
When is omitted in request payload, the previous nullable could remain
null and later cause failures during table change construction. This fix makes
the field behavior-safe and backward-compatible for callers that do not send .
### Does this PR introduce _any_ user-facing change?
Yes. request now supports omitted and defaults it to instead of failing.
### How was this patch tested?
- Added/ran unit test: .
- Verified with:
- To honour the JVM settings for this build a single-use Daemon process
will be forked. For more on this, please refer to
https://docs.gradle.org/8.2/userguide/gradle_daemon.html#sec:disabling_the_daemon
in the Gradle documentation.
Daemon will be stopped at the end of the build
Skipping filesystem-fuse module since enableFuse is set to false
> Configure project :
checkContainerRunning failed: Process 'command 'bash'' finished with
non-zero exit value 1
Current docker context is: orbstack
------------------ Check Docker environment ---------------------
Docker server status ............................................ [running]
mac-docker-connector status .....................................
[[31mstop[0m]
OrbStack status ................................................. [yes]
Using Docker container to run all tests ......................... [embedded
test]
-----------------------------------------------------------------
> Configure project :bundles
null
> Configure project :core
Kotlin DSL property assignment is an incubating feature.
> Task :spotlessInternalRegisterDependencies UP-TO-DATE
> Task :api:processResources UP-TO-DATE
> Task :common:processResources UP-TO-DATE
> Task :api:spotlessJava UP-TO-DATE
> Task :api:spotlessJavaCheck UP-TO-DATE
> Task :common:spotlessJava UP-TO-DATE
> Task :common:spotlessJavaCheck UP-TO-DATE
> Task :common:spotlessKotlinGradle UP-TO-DATE
> Task :api:spotlessKotlinGradle UP-TO-DATE
> Task :common:spotlessKotlinGradleCheck UP-TO-DATE
> Task :api:spotlessKotlinGradleCheck UP-TO-DATE
> Task :common:spotlessCheck UP-TO-DATE
> Task :api:spotlessCheck UP-TO-DATE
> Task :common:processTestResources NO-SOURCE
> Task :api:compileJava UP-TO-DATE
> Task :api:classes UP-TO-DATE
> Task :api:jar UP-TO-DATE
> Task :common:compileJava UP-TO-DATE
> Task :common:classes UP-TO-DATE
> Task :common:compileTestJava UP-TO-DATE
> Task :common:testClasses UP-TO-DATE
> Task :common:test UP-TO-DATE
> Task :common:jacocoTestReport UP-TO-DATE
Deprecated Gradle features were used in this build, making it incompatible
with Gradle 9.0.
You can use '--warning-mode all' to show the individual deprecation warnings
and determine if they come from your own scripts or plugins.
For more on this, please refer to
https://docs.gradle.org/8.2/userguide/command_line_interface.html#sec:command_line_warnings
in the Gradle documentation.
BUILD SUCCESSFUL in 45s
17 actionable tasks: 17 up-to-date
--
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]
