Skip to content

Add pre-JSON MySQL databases to new schema collector#22570

Open
sethsamuel wants to merge 19 commits intoeric.weaver/DBMON-6038from
seth.samuel/DBMON-6140-handle-non-json-enabled-my-sql-databases-with-new-schema-collector
Open

Add pre-JSON MySQL databases to new schema collector#22570
sethsamuel wants to merge 19 commits intoeric.weaver/DBMON-6038from
seth.samuel/DBMON-6140-handle-non-json-enabled-my-sql-databases-with-new-schema-collector

Conversation

@sethsamuel
Copy link
Contributor

@sethsamuel sethsamuel commented Feb 6, 2026

What does this PR do?

Adds support for MySQL databases without JSON aggregation to the new schema collector.

Motivation

The new collector is more efficient and enables backend performance optimizations.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@datadog-official
Copy link
Contributor

datadog-official bot commented Feb 6, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 7998794 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

@sethsamuel sethsamuel changed the base branch from master to eric.weaver/DBMON-6038 February 6, 2026 16:22
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.21%. Comparing base (f8fd12d) to head (7998794).

Additional details and impacted files
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sethsamuel sethsamuel changed the title Rename databases_data.py to schemas_legacy.py Add pre-JSON MySQL databases to new schema collector Feb 9, 2026
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3c913fe431

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

…dle-non-json-enabled-my-sql-databases-with-new-schema-collector
@sethsamuel sethsamuel force-pushed the seth.samuel/DBMON-6140-handle-non-json-enabled-my-sql-databases-with-new-schema-collector branch from 1aacdf2 to d62780c Compare March 4, 2026 16:20
@sethsamuel sethsamuel requested a review from eric-weaver March 12, 2026 14:06
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