LuciferYang opened a new pull request, #13245: URL: https://github.com/apache/gravitino/pull/13245
### What changes were proposed in this pull request? After conversion, `readFrom` now returns the `defaultValue` when the converted result is `null` and a non-null default exists, matching the absent-key path. String entries and no-default entries are unchanged. ### Why are the changes needed? `readFrom` only consulted the default for an absent key, so a present-but-blank value converted to `null` for the typed converters and bypassed the default, making every typed consumer NPE on unboxing. Fix: #13244 ### Does this PR introduce _any_ user-facing change? No API change. A present-but-blank config value now falls back to its configured default for int/long/double/boolean entries instead of returning `null`. String entries (where blank is valid) and entries with no default are unchanged. ### How was this patch tested? Added assertions in `TestConfigEntry`, which pin that a present-but-blank value falls back to the non-null default for the typed converters; they fail on the pre-fix tree and pass after the fix. -- 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]
