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

Revert "[DBMON-2989] report sql obfuscation error count (#15990)" #16493

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

lu-zhengda
Copy link
Contributor

What does this PR do?

Backport #16439

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Changelog entries must be created for modifications to shipped code
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

Copy link

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

@lu-zhengda lu-zhengda marked this pull request as ready for review December 21, 2023 15:35
@lu-zhengda lu-zhengda requested review from a team as code owners December 21, 2023 15:35
Copy link

codecov bot commented Dec 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (7.50.x@6ba77bb). Click here to learn what that means.

Additional details and impacted files
Flag Coverage Δ
mysql 87.45% <0.00%> (?)
postgres 91.91% <0.00%> (?)
sqlserver 89.91% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Copy link

Test Results

     48 files       48 suites   1h 15m 31s ⏱️
   649 tests    635 ✔️   14 💤 0
5 774 runs  5 475 ✔️ 299 💤 0

Results for commit c0492e4.

@iliakur iliakur merged commit 9b86463 into 7.50.x Dec 21, 2023
19 of 24 checks passed
@iliakur iliakur deleted the zhengda.lu/backport-16439 branch December 21, 2023 20:57
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