Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhanced monitoring when KMLC is batch listener #3047

Conversation

Wzy19930507
Copy link
Contributor

Because of observation does not support batch listener, add properties observationEnabled replace this.containerProperties.isObservationEnabled() for KafkaMessageListenerContainer.ListenerConsumer, when batch listener set false to observationEnabled.

  • add notice in micrometer.adoc
  • add observationEnabled replace containerProperties.isObservationEnabled() in KafkaMessageListenerContainer.ListenerConsumer

because of observation does not support batch listener, add properties `observationEnabled` replace `this.containerProperties.isObservationEnabled()` for `KafkaMessageListenerContainer.ListenerConsumer`, when batch listener set false to `observationEnabled`.

* add notice in micrometer.adoc
* add observationEnabled replace containerProperties.isObservationEnabled() in KafkaMessageListenerContainer.ListenerConsumer
@sobychacko sobychacko added this to the 3.2.0-M2 milestone Feb 21, 2024
@sobychacko sobychacko merged commit 6f32959 into spring-projects:main Feb 21, 2024
3 checks passed
sobychacko pushed a commit that referenced this pull request Feb 21, 2024
Fixes: #3047

* because of observation does not support batch listener, add properties `observationEnabled` and replace
  `this.containerProperties.isObservationEnabled()` for `KafkaMessageListenerContainer.ListenerConsumer`, when
  batch listener set false to `observationEnabled`.
* add notice in micrometer.adoc
* add observationEnabled and replace containerProperties.isObservationEnabled() in
  KafkaMessageListenerContainer.ListenerConsumer

**Auto-cherry-pick to `3.1.x`**
@Wzy19930507 Wzy19930507 deleted the batch_listener_not_support_observation branch February 22, 2024 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants