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 class name into Racecar instrumentation #1354

Open
ErvalhouS opened this issue Jan 30, 2023 · 0 comments
Open

Add class name into Racecar instrumentation #1354

ErvalhouS opened this issue Jan 30, 2023 · 0 comments

Comments

@ErvalhouS
Copy link

We're actively using our Racecar instrumentation on production, it captures what we need in terms of grouped spans into transactions.

There's a difficulty looking into a specific consumer's code performance on apps that have multiple consumers/producers. That is due to the fact that the instrumentation does transaction naming as follows: process_batch, process_message, deliver_message

The ideal scenario for transaction naming in that instrumentation would be to include the class that called those methods, in a similar way as what we currently have on controller's instrumentation. Something on the lines of "#{klass}#process_batch" i.e: DopeConsumer#process_batch or DopeProducer#deliver_message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant