-
Notifications
You must be signed in to change notification settings - Fork 3.3k
[receiver/azureeventhub] Promote azureeventhub receiver to beta #46037
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
base: main
Are you sure you want to change the base?
Conversation
|
|
||
| This feature gate controls which Azure Event Hubs client library is used by the receiver. When enabled (the default), the receiver uses the official [Azure SDK for Go](https://github.com/Azure/azure-sdk-for-go) `azeventhubs` library. This library provides improved performance, better error handling, and is the recommended choice. | ||
|
|
||
| The feature gate is now stable and enabled by default. It will be removed in v0.144.0, after which the `azeventhubs` library will be the only supported option. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe remove it? https://github.com/open-telemetry/opentelemetry-collector-contrib/releases/tag/v0.145.0 is out
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I remove this full section or just the explanation part? I am unaware of the process of what happens to a feature flag once it is in the stable state. Can the flag just be removed entirely from the code?
| ### Checkpoint Persistence | ||
|
|
||
| - Without a configured [storage extension], checkpoint state is stored only in memory. If the collector restarts, the receiver will resume from the latest offset, potentially missing messages or reprocessing already-consumed messages. | ||
| - When using storage, ensure the storage directory is persistent across restarts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
storage can be redis or db based, so not sure the storage directory advice is necessary
Description
README.mdREADME.mdlogs/metrics/tracesto betaLink to tracking issue
Fixes #41661