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

Fix queue length documentation #4872

Merged
merged 4 commits into from
Jul 26, 2024
Merged

Fix queue length documentation #4872

merged 4 commits into from
Jul 26, 2024

Conversation

tommyzli
Copy link
Contributor

The docs refer to the collector's queue as a retry queue, when it's actually a sending queue

@tommyzli tommyzli requested a review from a team July 23, 2024 23:41
@svrnm svrnm requested review from a team and evan-bradley and removed request for a team July 24, 2024 09:58
Copy link
Contributor

@evan-bradley evan-bradley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @tommyzli. Could you also update the metric definitions under the "basic-level metrics" section?

@tommyzli tommyzli requested a review from evan-bradley July 25, 2024 14:45
@svrnm svrnm self-assigned this Jul 25, 2024
@svrnm svrnm requested review from svrnm and a team July 25, 2024 17:45
@opentelemetrybot opentelemetrybot requested review from a team and TylerHelmuth and removed request for a team July 25, 2024 21:08
@chalin
Copy link
Contributor

chalin commented Jul 26, 2024

All checks are passing except for the known component-owners issue. I'd merge, but given that it's assigned to you @svrnm, I'll let you handle it.

@svrnm svrnm removed their assignment Jul 26, 2024
@svrnm
Copy link
Member

svrnm commented Jul 26, 2024

All checks are passing except for the known component-owners issue. I'd merge, but given that it's assigned to you @svrnm, I'll let you handle it.

The assignment was the outcome of a semi-failed run of the component owners. But yeah going to merge this.

@svrnm svrnm merged commit f172773 into open-telemetry:main Jul 26, 2024
16 of 17 checks passed
@svrnm
Copy link
Member

svrnm commented Jul 26, 2024

thank you @tommyzli !

@tommyzli tommyzli deleted the patch-2 branch July 26, 2024 15:19
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.

5 participants