hutiefang76 opened a new pull request, #11991:
URL: https://github.com/apache/seatunnel/pull/11991

   ### Purpose of this pull request
   
   Avoid a `/metrics` scrape returning HTTP 500 when the coordinator becomes 
unavailable between the existing readiness check and a collector lookup during 
startup or re-election. The affected collectors now skip that scrape only for 
`SeaTunnelEngineRetryableException`; unexpected failures still propagate.
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. During the short coordinator startup/re-election window, job and 
request-slot metric families may be absent from a scrape instead of causing the 
entire `/metrics` endpoint to return 500.
   
   ### How was this patch tested?
   
   - Added readiness-race regression coverage for `JobMetricExports` and 
`RequestSlotOperationExports`.
   - `TelemetryCollectorCoordinatorGuardTest`: 17 tests passed, including both 
new race cases.
   - Verified the affected engine common/core/server module chain with Maven 
`verify -DskipTests`.
   
   ### Check list
   
   * [x] No new Jar binary package is added.
   * [x] Documentation is not needed for this internal readiness-race fix.
   * [x] No incompatible change is introduced.
   * [x] This change does not modify connector 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]

Reply via email to