OIiveirra opened a new pull request, #68697:
URL: https://github.com/apache/doris/pull/68697
## Summary
```text
ALTER CATALOG
-> validate submitted properties
-> reject an explicitly submitted empty dfs.nameservices
-> otherwise bind the merged catalog properties
```
Preserve runtime compatibility for legacy HMS catalogs whose persisted
`dfs.nameservices` value is empty while keeping CREATE and ALTER validation
strict for newly submitted values. Unrelated ALTER statements now proceed on
those legacy catalogs.
## Evidence
- **Before:** `ALTER CATALOG ... SET PROPERTIES ("test_connection"="false")`
failed when the existing catalog contained `dfs.nameservices=,`.
- **After:** Only an ALTER that explicitly submits an empty
`dfs.nameservices` is rejected; unrelated ALTERs retain the legacy catalog
value and pass validation.
- **Tests:** Added FE coverage for legacy empty values and explicit
CREATE/ALTER rejection cases. The focused FE UT reached test execution but the
local Mockito inline mock maker could not self-attach in this environment;
FE/HDFS unit coverage for the related runtime compatibility path passes.
## Merge Danger
**Door:** two-way
The change is limited to HMS catalog property validation and is
independently revertible.
**Blast Radius:** catalog-validation
--
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]
