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.

Sign up for GitHub

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

Jump to bottom

Add perf test for exporting request telemetry #35223

Merged
merged 7 commits into from
Apr 3, 2023
Merged

Add perf test for exporting request telemetry #35223

merged 7 commits into from
Apr 3, 2023

Conversation

vishweshbankwar
Copy link
Contributor

@vishweshbankwar vishweshbankwar commented Mar 29, 2023 •
edited
Loading

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@github-actions github-actions bot added the Monitor - Exporter Monitor OpenTelemetry Exporter label Mar 29, 2023
@vishweshbankwar vishweshbankwar marked this pull request as ready for review March 30, 2023 20:54
TimothyMothra
TimothyMothra approved these changes Mar 30, 2023
View reviewed changes
Copy link
Contributor

@TimothyMothra TimothyMothra left a comment

Choose a reason for hiding this comment

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

LGTM.

What kind of output does this generate?

rajkumar-rangaraj
rajkumar-rangaraj approved these changes Mar 30, 2023
View reviewed changes
Copy link
Contributor

@rajkumar-rangaraj rajkumar-rangaraj left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM.

Document the output from one of the run of your local environment to PR description.

@vishweshbankwar
Copy link
Contributor Author

LGTM.

What kind of output does this generate?

Added in PR desc

@vishweshbankwar vishweshbankwar merged commit b4c0695 into main Apr 3, 2023
@vishweshbankwar vishweshbankwar deleted the vibankwa/perf-test-distro branch April 3, 2023 16:53
@TimothyMothra TimothyMothra mentioned this pull request Apr 11, 2023
Closed
@rajkumar-rangaraj rajkumar-rangaraj mentioned this pull request May 26, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TimothyMothra TimothyMothra TimothyMothra approved these changes

@rajkumar-rangaraj rajkumar-rangaraj rajkumar-rangaraj approved these changes

@cijothomas cijothomas Awaiting requested review from cijothomas

@reyang reyang Awaiting requested review from reyang

Assignees
No one assigned
Labels
Monitor - Exporter Monitor OpenTelemetry Exporter
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vishweshbankwar @azure-sdk @rajkumar-rangaraj @TimothyMothra