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

Clean up database systems #1120

Merged
merged 5 commits into from
Jun 7, 2024
Merged

Conversation

lmolkova
Copy link
Contributor

@lmolkova lmolkova commented Jun 4, 2024

Fixes #1023

Changes

Merge requirement checklist

@lmolkova lmolkova requested review from a team June 4, 2024 01:36
@lmolkova
Copy link
Contributor Author

lmolkova commented Jun 4, 2024

I did not find any usage of cloudscape, coldfusion or firstsql in OpenTelemetry repositories.
Checked cache in java, python and JS instrumentations - no mentions.

mssqlcompact is still used by .NET OpenTelemetry.Instrumentation.EntityFrameworkCore, but the corresponding packages for mssqlcompact are deprecated (SqlServerCompact40, SqlServerCompact35) and have <50 downloads per day.
The other_sql can be used as a fallback option.

model/registry/db.yaml Outdated Show resolved Hide resolved
docs/attributes-registry/db.md Outdated Show resolved Hide resolved
@lmolkova lmolkova force-pushed the db-system-clean-up branch 2 times, most recently from 5aa261b to e051252 Compare June 5, 2024 19:11
model/registry/db.yaml Outdated Show resolved Hide resolved
model/registry/db.yaml Outdated Show resolved Hide resolved
@lmolkova lmolkova merged commit ca6e1c5 into open-telemetry:main Jun 7, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Database: review db.system list
4 participants