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

Upgrade app insights to latest #4127

Merged
merged 4 commits into from
Nov 12, 2024
Merged

Upgrade app insights to latest #4127

merged 4 commits into from
Nov 12, 2024

Conversation

adusumillipraveen
Copy link
Contributor

Sampling changes done in #4123 haven't shown any impact, most likely due to some bug fixes upstream. microsoft/ApplicationInsights-Java#3588 https://github.com/microsoft/ApplicationInsights-Java/pull/3578/files

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@adusumillipraveen adusumillipraveen requested a review from a team as a code owner November 11, 2024 09:44
Copy link

CCD diff report

No change

@adusumillipraveen adusumillipraveen merged commit c564028 into master Nov 12, 2024
12 checks passed
@adusumillipraveen adusumillipraveen deleted the upgradeai branch November 12, 2024 16:13
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.

4 participants