KyleLin0927 opened a new pull request, #7238: URL: https://github.com/apache/gravitino/pull/7238
### What changes were proposed in this pull request? - Added a `filterAndValidateStorageLocations` private method in `createMultipleLocationFileset` to remove null, empty, or whitespace-only entries from the user’s storageLocations. - After filtering, if no user locations remain, fall back to schema- or catalog-level location properties. If those are also unset, trigger the existing “Storage location must be set…” validation error (instead of an NPE). - Updated the web UI so it no longer auto-assigns `DEFAULT_LOCATION_NAME` when the user’s StorageLocation is null, empty, or whitespace—preventing invalid defaults from ever reaching the API. ### Why are the changes needed? Fixes #7190 ### Does this PR introduce any user-facing change? No. APIs and behavior are unchanged, only error messages for invalid input are clearer. ### How was this patch tested? - Added new unit tests. - Verified with `./gradlew clean build`. -- 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]
