-
Notifications
You must be signed in to change notification settings - Fork 296
Update emitter to 0.19.0 #2822
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
Update emitter to 0.19.0 #2822
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the @azure-tools/typespec-rust
dependency from version 0.18.0 to 0.19.0 in the emitter package configuration.
- Updates TypeSpec Rust tools dependency to version 0.19.0
Files not reviewed (1)
- eng/emitter-package-lock.json: Language not supported
Does this obsolete #2816? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM for Storage.
One aside, are the tracing names i.e.[tracing::function("Storage.Blob.Container.Blob.AppendBlob.appendBlock")]
rendered as such, or how do they show up for the customer?
If the customer has enabled distributed tracing in their This can be overwritten for wrapped clients if necessary (and it may be necessary/desirable for storage clients). Longer term, the plan is to add a knob that client teams who write wrapped clients can switch to disable the distributed tracing macros in favor of client specific implementations. See this for more information on the distributed tracing infrastructure. |
No description provided.