YuF-9468 opened a new pull request, #10230:
URL: https://github.com/apache/gravitino/pull/10230
## Summary
Fixes #10226.
`FunctionEntity.Builder.build()` validates required fields through
`fields()`, but `namespace` was missing from that map, allowing invalid
entities to be built with null namespace. This PR marks namespace as required
and includes it in validation.
## Changes
- Added `NAMESPACE` as a required `Field` in `FunctionEntity`.
- Added `namespace` to `FunctionEntity#fields()` so `build()` validation
enforces it.
- Added `TestFunctionEntity` with:
- a regression test that `build()` throws when namespace is missing;
- a positive test that build succeeds when namespace is provided.
## Validation
- Added/updated unit tests for the changed behavior.
- Could not run local Gradle tests in this environment because
Java/JAVA_HOME is unavailable.
--
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]
