sdj3261 opened a new pull request, #7526:
URL: https://github.com/apache/gravitino/pull/7526
[#7523] fix(config): avoid logging null value in ConfigEntry writes
- Prevents using null as a log parameter when writing config entries.
- Logs the config key instead when the string-converted value is null.
- Improves log clarity and avoids potential confusion during debugging.
### What changes were proposed in this pull request?
This PR fixes an issue in `ConfigEntry.writeTo()`.
Previously, if the converted string value was `null`, the log message would
print `"null"` as the config key, leading to confusion.
Now, the code ensures that the actual config key is logged instead,
preventing null values from appearing in logs.
### Why are the changes needed?
Logging a null value as a key can cause confusion during debugging and make
logs misleading.
This fix improves log clarity and ensures that null values are not
accidentally logged or stored.
Fixes #7523
### Does this PR introduce _any_ user-facing change?
No user-facing changes. Only internal logging behavior has been improved.
### How was this patch tested?
- Unit tests were added and executed to ensure that when the string value is
null,
the key is correctly logged instead.
- Integration tests were not run locally due to Docker environment
constraints
and will be verified via CI.
--
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]
