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

Add tracing, logging, and caching to the plugin #412

Closed
craigtmoore opened this issue Jun 10, 2024 · 1 comment · Fixed by #413
Closed

Add tracing, logging, and caching to the plugin #412

craigtmoore opened this issue Jun 10, 2024 · 1 comment · Fixed by #413
Labels
enhancement New feature or request

Comments

@craigtmoore
Copy link
Contributor

What feature do you want to see added?

In order to measure the performance of the plugin when accessing the database, we should add open telemetry tracing and logging. In addition, there are opportunities to introduce caching so that the plugin doesn't have to access the database each time it needs to access data. instead it can store the data temporarily in a cache to improve access times.

Upstream changes

No response

Are you interested in contributing this feature?

yes, I plan to create a pull request soon to facilitate these changes.

@cyrille-leclerc
Copy link
Contributor

@craigtmoore @timja FYI we are introducing an opentelemetry-api plugin to cleanup your dependencies:

I also proposed an enhancement of the SQL spans:

cc @kuisathaverat

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants