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

First metric and tracing implementation #373

Merged
merged 5 commits into from
Dec 14, 2021
Merged

First metric and tracing implementation #373

merged 5 commits into from
Dec 14, 2021

Conversation

nappa85
Copy link
Contributor

@nappa85 nappa85 commented Dec 9, 2021

As discussed on #372, I've done a first, minimal, implementation for tracing and metric support.
I'm actually using this version on my environment and it seems to works like a charm, especially the metric part.
For the Tracing part, I've instrumented a minimal part of crate methods, and I've decided to instrument it as level trace because I think a library crate should not be "too loud".
Maybe we could control that behavior with a feature if you prefer

@tyt2y3
Copy link
Member

tyt2y3 commented Dec 10, 2021

Awesome! Thank you for the big contribution again.
Looks really good so far.

I think tracing has log output support, so we might want to put that shim behind a 'use-log' feature flag.

I have no opinion on the trace level, it's good for now.

@nappa85
Copy link
Contributor Author

nappa85 commented Dec 10, 2021

I think tracing has log output support, so we might want to put that shim behind a 'use-log' feature flag.

What do you mean by "log output support"?
To output log messages you need a subscriber, if you look the crate is in dev-dependencies in place of env-logger to allow test logging to work as before

@tyt2y3
Copy link
Member

tyt2y3 commented Dec 10, 2021

What do you mean by "log output support"?

I found it, I mean this https://docs.rs/tracing/0.1.29/tracing/index.html#log-compatibility
Seems like users can enable that feature flag on their own, so we don't have to do anything for it )

Copy link
Member

@tyt2y3 tyt2y3 left a comment

Choose a reason for hiding this comment

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

Looks good to me. @billy1624 thoughts?

@tyt2y3
Copy link
Member

tyt2y3 commented Dec 12, 2021

We will release this in 0.5.0

@tyt2y3 tyt2y3 merged commit 7da5b6b into SeaQL:master Dec 14, 2021
billy1624 added a commit to SeaQL/seaql.github.io that referenced this pull request Dec 23, 2021
tyt2y3 pushed a commit to SeaQL/seaql.github.io that referenced this pull request Dec 31, 2021
docs: rework `ActiveValue` (SeaQL/sea-orm#340)

docs: schema_name will always be applied (SeaQL/sea-orm#370)

docs: `ActiveModelBehavior::after_save` takes `Model` (SeaQL/sea-orm#339)

docs: `Schema::create_enum_from_active_enum` (SeaQL/sea-orm#348)

docs: `ModelTrait::delete` (SeaQL/sea-orm#396)

docs: codegen SQLite (SeaQL/sea-orm#386)

docs: unsigned integers (SeaQL/sea-orm#397)

docs: debug log (SeaQL/sea-orm#373)

blog: draft "What's new in SeaORM 0.5.0"

docs: revert `save` to return Self (SeaQL/sea-orm@d5c9c65)

Revert "docs: unsigned integers (SeaQL/sea-orm#397)"

This reverts commit ac6d49e.

docs: exclude SeaQL/sea-orm#397 in 0.5.0 blog post

docs: update landing page examples

docs: edit

docs: edit

docs: fix 0.4.x docs link

build: bump docusaurus version to 2.0.0-beta.14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants