Describe the Bug
TieredStoreMetricsManager catches ConsumeQueueException while collecting the dispatchBehind and dispatchLatency gauges, then silently ignores the exception. The affected callbacks are in initMetrics().
When either collection path fails, the metric has no measurement for that collection cycle and the broker logs do not show the failure. Operators cannot distinguish a zero metric value from an unsuccessful collection attempt.
PopMetricsManager logs ConsumeQueueException with log.error(message, e) for its gauge callbacks. Applying the same handling here keeps the metric callbacks non-disruptive while making failures observable.
Expected Behavior
Tiered Store metric collection failures should be logged with the exception details.
Additional Context
The proposed change only adds error logging in the two existing catch (ConsumeQueueException e) blocks. Normal metric collection and message processing behavior remain unchanged.
Describe the Bug
TieredStoreMetricsManagercatchesConsumeQueueExceptionwhile collecting thedispatchBehindanddispatchLatencygauges, then silently ignores the exception. The affected callbacks are ininitMetrics().When either collection path fails, the metric has no measurement for that collection cycle and the broker logs do not show the failure. Operators cannot distinguish a zero metric value from an unsuccessful collection attempt.
PopMetricsManagerlogsConsumeQueueExceptionwithlog.error(message, e)for its gauge callbacks. Applying the same handling here keeps the metric callbacks non-disruptive while making failures observable.Expected Behavior
Tiered Store metric collection failures should be logged with the exception details.
Additional Context
The proposed change only adds error logging in the two existing
catch (ConsumeQueueException e)blocks. Normal metric collection and message processing behavior remain unchanged.