-
Notifications
You must be signed in to change notification settings - Fork 154
Feature request: Allow users to specify segment names for captured methods #1084
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
Comments
Thanks for opening this issue. I agree with you about the benefits highlighted and this is certainly a feature we could consider adding in the future. |
@dreamorosi Just as an aside, here's another interesting edge case where specifying In this case, the tracer tries to derive the name of the segment from the anonymous function returned by the And here's a workaround for now: Edit: I've opened an issue for this edge case #1093, as I think it's a fixable problem. |
|
Description of the feature request
Problem statement
Tracer's capture decorators currently try to determine the name of the segment from the function it decorates. While this is fine for a small project, for a larger app, auto-generated names like
### render
are not descriptive enough to identify which part of the code is represented by the segment.Summary of the feature
I'd like the ability to specify the segment name when I capture methods.
Code examples
Benefits for you and the wider AWS community
Allows developers to more clearly identify the sources of segments in larger codebases. This would make navigating a trace in the X-Ray console easier.
Describe alternatives you've considered
I've been putting metadata in the segments to help identify the source code associated with a segment, but this requires a lot of clicking in the X-Ray console to find what I'm looking for.
Additional context
In my example above, I named my segments to reference the corresponding concrete class names. In an ideal world, I'd want the automatic naming to replicate this. But I was unsure of two things:
Related issues, RFCs
None
The text was updated successfully, but these errors were encountered: