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

Support newrelic_rpm >= 6.0.0 #11

Merged
merged 4 commits into from
Aug 5, 2020
Merged

Support newrelic_rpm >= 6.0.0 #11

merged 4 commits into from
Aug 5, 2020

Conversation

kaorimatz
Copy link
Contributor

@kaorimatz kaorimatz commented Aug 3, 2020

I had a quick chat with @uzzu and took over from where he left off with #8.

Tracer.start_transaction_or_segment is available since 6.0.0.351: newrelic/newrelic-ruby-agent@0de064c

@eagletmt
Copy link
Member

eagletmt commented Aug 4, 2020

Is it impossible to keep compatibility between newrelic_rpm < v6.0.0 and newrelic_rpm >= v6.0.0?
It's OK to drop support for newrelic_rpm < v6.0.0 in a future release, but it looks troublesome if there was no way other than upgrading/downgrading both griffin-interceptors and newrelic_rpm simultaneously.

@kaorimatz kaorimatz changed the title Use NewRelic::Agent::Tracer Support newrelic_rpm >= 6.0.0 Aug 4, 2020
@kaorimatz
Copy link
Contributor Author

kaorimatz commented Aug 4, 2020

Good point. Let's keep support for newrelic_rpm < 6.0.0 for now. Could you have another look please?

@kaorimatz
Copy link
Contributor Author

Thank you for the review!

@kaorimatz kaorimatz merged commit 1c8a358 into cookpad:master Aug 5, 2020
@kaorimatz kaorimatz deleted the use-new-relic-agent-tracer branch August 5, 2020 06:49
@kaorimatz
Copy link
Contributor Author

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.

None yet

3 participants