realestate-com-au / newrelic-sequel

newrelic-sequel integration plugin
5 stars 5 forks source link

Not compatible with newrelic_rpm 3.9.0.229 #2

Open bethesque opened 10 years ago

bethesque commented 10 years ago

The new Newrelic::Agent seems to not have the method is_execution_traced? any more.

undefined method `is_execution_traced?' for NewRelic::Agent:Module
"/usr/local/var/rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/newrelic-sequel-0.0.5/lib/newrelic_sequel/sequel.rb:76:in `log_duration_with_newrelic_instrumentation'",
"/usr/local/var/rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/sequel-4.11.0/lib/sequel/database/logging.rb:42:in `log_yield'",
"/usr/local/var/rbenv/versions/1.9.3-p484bet/lib/ruby/gems/1.9.1/gems/newrelic_rpm-3.9.0.229/lib/sequel/extensions/newrelic_instrumentation.rb:43:in `log_yield'",
"/usr/local/var/rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/sequel-4.11.0/lib/sequel/adapters/tinytds.rb:133:in `log_connection_execute'",
"/usr/local/var/rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/sequel-4.11.0/lib/sequel/database/connecting.rb:255:in `valid_connection?'",
"/usr/local/var/rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/sequel-4.11.0/lib/sequel/extensions/connection_validator.rb:88:in `acquire'",
"/usr/local/var/rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/sequel-4.11.0/lib/sequel/connection_pool/threaded.rb:94:in `hold'",
"/usr/local/var/rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/sequel-4.11.0/lib/sequel/database/connecting.rb:229:in `synchronize'",
"/Users/some-user/project/db.rb:62:in `db_health_check'"

We have worked around this by locking the newrelic_rpm version to 3.8.1.221

bethesque commented 10 years ago

Poke?

rosenfeld commented 10 years ago

Same issue here