henrib opened a new pull request, #6937:
URL: https://github.com/apache/hive/pull/6937
### What changes were proposed in this pull request?
- `HMSReadinessHealthIndicator` reports only `UP`/`DOWN`. The HMS Thrift
URIs, the warehouse
location and the raw exception message are no longer included in the
health response;
failure causes are logged instead (message at WARN, stack trace at DEBUG).
- `application.yml`: `management.endpoint.health.show-details` changes from
`always` to
`never`, and a comment documents `management.server.port` for serving the
actuator
endpoints on a separate port.
### Why are the changes needed?
The health endpoint is used by Kubernetes probes, which only need the
status. Server
configuration and error details do not belong in that response.
### Does this PR introduce _any_ user-facing change?
Yes. `/actuator/health` (and the liveness/readiness groups) of the
standalone REST catalog
server return only the status, without a `details`/`components` breakdown.
Probe behaviour
is unchanged. Details can be re-enabled with
`-Dmanagement.endpoint.health.show-details=always`, but the HMS indicator no
longer
reports URIs, warehouse or error text.
### How was this patch tested?
New `TestHMSReadinessHealthIndicator`: DOWN when Thrift URIs are not
configured, DOWN when
HMS is unreachable, UP against a running HMS; each case asserts no details
are returned.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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]
