tingyun_rpm 1.2.0 → 1.3.0

data/lib/ting_yun/ting_yun_service/upload_service.rb

@@ -2,6 +2,8 @@
 require 'ting_yun/metrics/metric_spec'
 require 'ting_yun/metrics/metric_data'
 require 'ting_yun/support/serialize/encodes'
+require 'ting_yun/support/quantile_p2'
+require 'json'
 
 module TingYun
   class TingYunService
@@ -20,9 +22,8 @@ module TingYun
         TingYun::Support::Serialize::Encoders::Json
       end
 
-      def metric_data(stats_hash)
-
-        action_array, adpex_array, general_array, components_array, errors_array = build_metric_data_array(stats_hash)
+      def metric_data(stats_hash, base_quantile_hash)
+        action_array, adpex_array, general_array, components_array, errors_array = build_metric_data_array(stats_hash, base_quantile_hash)
 
         upload_data = {
             :type => 'perfMetrics',
@@ -35,7 +36,9 @@ module TingYun
             :general => general_array,
             :errors  => errors_array
         }
+        upload_data.merge!(:config => {"nbs.quantile" => TingYun::Agent.config[:'nbs.quantile']}) if TingYun::Agent.config[:'nbs.quantile']
         result = invoke_remote(:upload, [upload_data])
+        self.quantile_cache = {}
         fill_metric_id_cache(result)
         result
       end
@@ -44,12 +47,15 @@ module TingYun
       # The collector wants to recieve metric data in a format that's different
       # # from how we store it inte -nally, so this method handles the translation.
       # # It also handles translating metric names to IDs using our metric ID cache.
-      def build_metric_data_array(stats_hash)
+      def build_metric_data_array(stats_hash, base_quantile_hash)
         action_array = []
         adpex_array = []
         general_array = []
         components_array = []
         errors_array = []
+
+        calculate_quantile(base_quantile_hash.hash)
+
         stats_hash.each do |metric_spec, stats|
 
           # Omit empty stats as an optimization
@@ -57,17 +63,11 @@ module TingYun
             metric_id = metric_id_cache[metric_spec.hash]
 
             if metric_spec.name.start_with?('WebAction','BackgroundAction')
-              action_array << TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id)
+              action_array << generate_action(metric_spec, stats, metric_id)
             elsif metric_spec.name.start_with?('Apdex')
               adpex_array << TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id)
             elsif metric_spec.name.start_with?('Errors') && metric_spec.scope.empty?
               errors_array << TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id)
-            elsif metric_spec.name.start_with?('Database','View','MongoDB','Redis','Memcached','External','Nested', 'CPU', 'Memory', 'WebFrontend','Rake')
-              if metric_spec.scope.empty?
-                general_array << TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id)
-              else
-                components_array << TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id) unless metric_spec.name.start_with?('External/NULL')
-              end
             elsif metric_spec.name.start_with?('cross_app')
               external =  metric_spec.name.split(';')
               if metric_spec.scope.empty?
@@ -79,14 +79,40 @@ module TingYun
                 metric_spec.calleeName = external[2]
                 components_array << TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id)
               end
-
+            else
+              if metric_spec.scope.empty?
+                general_array << TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id)
+              else
+                components_array << TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id) unless metric_spec.name.start_with?('External/NULL')
+              end
             end
+
           end
         end
 
         [action_array, adpex_array, general_array, components_array, errors_array]
       end
 
+      def generate_action(metric_spec, stats, metric_id)
+        if !TingYun::Support::QuantileP2.support? || metric_spec.name.start_with?('BackgroundAction')
+          TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id)
+        else
+          quantile = self.quantile_cache[metric_spec.full_name] || []
+          TingYun::Metrics::MetricData.new(metric_spec, stats, metric_id, quantile)
+        end
+      end
+
+      def calculate_quantile(base)
+        if TingYun::Support::QuantileP2.support?
+          quantile = TingYun::Agent.config[:'nbs.quantile']
+          base.each do |action_name, base_list|
+            qm = TingYun::Support::QuantileP2.new(JSON.parse(quantile).map{|i| i.to_f/100})
+            base_list.each{ |l| qm.add(l) }
+            self.quantile_cache[action_name] = qm.markers
+          end
+        end
+      end
+
 
       # takes an array of arrays of spec and id, adds it into the
       # metric cache so we can save the collector some work by

data/lib/ting_yun/ting_yun_service.rb

@@ -29,7 +29,8 @@ module TingYun
                   :metric_id_cache,
                   :applicationId,
                   :ssl_cert_store,
-                  :shared_tcp_connection
+                  :shared_tcp_connection,
+                  :quantile_cache
 
 
     def initialize(license_key=nil)
@@ -39,6 +40,7 @@ module TingYun
       @data_version = TingYun::VERSION::STRING
       @marshaller =TingYun::Support::Serialize::JsonMarshaller.new
       @metric_id_cache = {}
+      @quantile_cache = {}
     end
 
     def connect(settings={})
@@ -61,6 +63,7 @@ module TingYun
       @applicationId = nil
       @appSessionKey = nil
       @metric_id_cache = {}
+      @quantile_cache = {}
       close_shared_connection
     end
 
@@ -121,11 +124,5 @@ module TingYun
       raise error
     end
 
-
-
-
-
-
-
   end
 end

data/lib/ting_yun/version.rb

@@ -6,12 +6,14 @@ module TingYun
 
     MAJOR = 1
 
-    MINOR = 2
+    MINOR = 3
 
     TINY = 0
 
 
 
+
+
     STRING = [MAJOR, MINOR, TINY].compact.join('.')
   end
 end

data/lib/tingyun_rpm.rb

@@ -23,10 +23,8 @@
 
 require 'ting_yun/frameworks'
 
-# require 'pry'
 #if the agent had started in manual , then shouldn't start in auto again
 
-
 if defined?(Rails::VERSION)
   if Rails::VERSION::MAJOR.to_i >= 3
     module TingYun
@@ -48,5 +46,3 @@ if defined?(Rails::VERSION)
 else
   TingYun::Frameworks.init_start
 end
-
-

data/tingyun_rpm.gemspec

@@ -25,12 +25,12 @@ Gem::Specification.new do |s|
   s.require_paths = ['lib']
   s.rubygems_version = Gem::VERSION
 
-  s.add_development_dependency 'rake', '10.1.0'
+  s.add_development_dependency 'rake', '~> 10.1.0'
   s.add_development_dependency 'minitest', '~> 4.7'
-  s.add_development_dependency 'minitest-ci', '= 2.4.0'
+  s.add_development_dependency 'minitest-ci', '~> 2.4.0'
   s.add_development_dependency 'minitest-focus', '~> 1.1'
   s.add_development_dependency 'minitest-reporters', '0.14.24'
-  s.add_development_dependency 'simplecov', '= 0.10.0'
+  s.add_development_dependency 'simplecov', '~> 0.10.0'
   s.add_development_dependency 'mocha', '~> 0.13.0'
   s.add_development_dependency 'rails', '~> 3.2'
   s.add_development_dependency 'pry', '~> 0.9.12'

metadata

@@ -1,27 +1,27 @@
 --- !ruby/object:Gem::Specification
 name: tingyun_rpm
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors:
 - tingyun
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-08 00:00:00.000000000 Z
+date: 2017-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 10.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 10.1.0
 - !ruby/object:Gem::Dependency
@@ -42,14 +42,14 @@ dependencies:
   name: minitest-ci
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 2.4.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 2.4.0
 - !ruby/object:Gem::Dependency
@@ -84,14 +84,14 @@ dependencies:
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.10.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.10.0
 - !ruby/object:Gem::Dependency
@@ -270,7 +270,6 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".travis.yml"
-- CODE_OF_CONDUCT.md
 - Gemfile
 - Guardfile
 - LICENSE.txt
@@ -292,6 +291,7 @@ files:
 - lib/ting_yun/agent/collector/sql_sampler/sql_trace.rb
 - lib/ting_yun/agent/collector/sql_sampler/transaction_sql_data.rb
 - lib/ting_yun/agent/collector/stats_engine.rb
+- lib/ting_yun/agent/collector/stats_engine/base_quantile_hash.rb
 - lib/ting_yun/agent/collector/stats_engine/metric_stats.rb
 - lib/ting_yun/agent/collector/stats_engine/stats_hash.rb
 - lib/ting_yun/agent/collector/transaction_sampler.rb
@@ -318,7 +318,6 @@ files:
 - lib/ting_yun/agent/instance_methods/handle_errors.rb
 - lib/ting_yun/agent/instance_methods/start.rb
 - lib/ting_yun/agent/instance_methods/start_worker_thread.rb
-- lib/ting_yun/agent/method_tracer.rb
 - lib/ting_yun/agent/method_tracer_helpers.rb
 - lib/ting_yun/agent/threading/agent_thread.rb
 - lib/ting_yun/agent/transaction.rb
@@ -352,11 +351,19 @@ files:
 - lib/ting_yun/frameworks/rails.rb
 - lib/ting_yun/frameworks/rails3.rb
 - lib/ting_yun/frameworks/rails4.rb
+- lib/ting_yun/frameworks/rails5.rb
 - lib/ting_yun/frameworks/ruby.rb
 - lib/ting_yun/frameworks/sinatra.rb
 - lib/ting_yun/http/generic_request.rb
+- lib/ting_yun/http/http_client_request.rb
 - lib/ting_yun/http/net_http_request.rb
 - lib/ting_yun/instrumentation/active_record.rb
+- lib/ting_yun/instrumentation/bunny.rb
+- lib/ting_yun/instrumentation/data_mapper.rb
+- lib/ting_yun/instrumentation/grape.rb
+- lib/ting_yun/instrumentation/http_client.rb
+- lib/ting_yun/instrumentation/kafka.rb
+- lib/ting_yun/instrumentation/memcached.rb
 - lib/ting_yun/instrumentation/middleware_proxy.rb
 - lib/ting_yun/instrumentation/middleware_tracing.rb
 - lib/ting_yun/instrumentation/mongo.rb
@@ -367,23 +374,37 @@ files:
 - lib/ting_yun/instrumentation/rack.rb
 - lib/ting_yun/instrumentation/rails3/action_controller.rb
 - lib/ting_yun/instrumentation/rails3/action_view.rb
-- lib/ting_yun/instrumentation/rails4/action_controller_subscriber.rb
-- lib/ting_yun/instrumentation/rails4/action_view_subscriber.rb
-- lib/ting_yun/instrumentation/rails4/active_record_subscriber.rb
+- lib/ting_yun/instrumentation/rails4/action_controller.rb
+- lib/ting_yun/instrumentation/rails4/action_view.rb
+- lib/ting_yun/instrumentation/rails4/active_record.rb
+- lib/ting_yun/instrumentation/rails5/action_cable.rb
+- lib/ting_yun/instrumentation/rails5/action_controller.rb
+- lib/ting_yun/instrumentation/rails5/action_view.rb
+- lib/ting_yun/instrumentation/rails5/active_record.rb
 - lib/ting_yun/instrumentation/rails_middleware.rb
 - lib/ting_yun/instrumentation/rake.rb
 - lib/ting_yun/instrumentation/redis.rb
+- lib/ting_yun/instrumentation/sidekiq.rb
+- lib/ting_yun/instrumentation/sinatra/action.rb
+- lib/ting_yun/instrumentation/sinatra/view.rb
+- lib/ting_yun/instrumentation/support/action_cable_subscriber.rb
+- lib/ting_yun/instrumentation/support/action_controller_subscriber.rb
+- lib/ting_yun/instrumentation/support/action_view_subscriber.rb
 - lib/ting_yun/instrumentation/support/active_record_helper.rb
+- lib/ting_yun/instrumentation/support/active_record_subscriber.rb
 - lib/ting_yun/instrumentation/support/controller_instrumentation.rb
 - lib/ting_yun/instrumentation/support/database.rb
 - lib/ting_yun/instrumentation/support/event_formatter.rb
 - lib/ting_yun/instrumentation/support/evented_subscriber.rb
 - lib/ting_yun/instrumentation/support/external_error.rb
+- lib/ting_yun/instrumentation/support/external_helper.rb
 - lib/ting_yun/instrumentation/support/javascript_instrumentor.rb
+- lib/ting_yun/instrumentation/support/method_instrumentation.rb
 - lib/ting_yun/instrumentation/support/metric_translator.rb
 - lib/ting_yun/instrumentation/support/mongo_formatter.rb
 - lib/ting_yun/instrumentation/support/parameter_filtering.rb
 - lib/ting_yun/instrumentation/support/queue_time.rb
+- lib/ting_yun/instrumentation/support/sinatra_helper.rb
 - lib/ting_yun/instrumentation/support/split_controller.rb
 - lib/ting_yun/instrumentation/support/thrift_helper.rb
 - lib/ting_yun/instrumentation/support/transaction_namer.rb
@@ -412,6 +433,7 @@ files:
 - lib/ting_yun/support/library_detection.rb
 - lib/ting_yun/support/local_environment.rb
 - lib/ting_yun/support/path.rb
+- lib/ting_yun/support/quantile_p2.rb
 - lib/ting_yun/support/serialize/encodes.rb
 - lib/ting_yun/support/serialize/encoding_normalizer.rb
 - lib/ting_yun/support/serialize/json_marshaller.rb

data/CODE_OF_CONDUCT.md

@@ -1,13 +0,0 @@
-# Contributor Code of Conduct
-
-As contributors and maintainers of this project, we pledge to respect all people who contribute through reporting issues, posting feature requests, updating documentation, submitting pull requests or patches, and other activities.
-
-We are committed to making participation in this project a harassment-free experience for everyone, regardless of level of experience, gender, gender identity and expression, sexual orientation, disability, personal appearance, body size, race, ethnicity, age, or religion.
-
-Examples of unacceptable behavior by participants include the use of sexual language or imagery, derogatory comments or personal attacks, trolling, public or private harassment, insults, or other unprofessional conduct.
-
-Project maintainers have the right and responsibility to remove, edit, or reject comments, commits, code, wiki edits, issues, and other contributions that are not aligned to this Code of Conduct. Project maintainers who do not follow the Code of Conduct may be removed from the project team.
-
-Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by opening an issue or contacting one or more of the project maintainers.
-
-This Code of Conduct is adapted from the [Contributor Covenant](http://contributor-covenant.org), version 1.0.0, available at [http://contributor-covenant.org/version/1/0/0/](http://contributor-covenant.org/version/1/0/0/)

data/lib/ting_yun/agent/method_tracer.rb

@@ -1,256 +0,0 @@
-# encoding: utf-8
-
-# This file may be independently required to set up method tracing prior to
-# the full agent loading. In those cases, we do need at least this require to
-# bootstrap things.
-
-require 'ting_yun/frameworks' unless defined?(TingYun::Frameworks::Framework)
-require 'ting_yun/support/helper'
-require 'ting_yun/agent/method_tracer_helpers'
-
-module TingYun
-  module Agent
-
-    # This module contains class methods added to support installing custom
-    # metric tracers and executing for individual metrics.
-    #
-    # == Examples
-    #
-    # When the agent initializes, it extends Module with these methods.
-    # However if you want to use the API in code that might get loaded
-    # before the agent is initialized you will need to require
-    # this file:
-    #
-    #     require 'ting_yun/agent/method_tracer'
-    #     class A
-    #       include TingYun::Agent::MethodTracer
-    #       def process
-    #         ...
-    #       end
-    #       add_method_tracer :process
-    #     end
-    #
-    # To instrument a class method:
-    #
-    #     require 'ting_yun/agent/method_tracer'
-    #     class An
-    #       def self.process
-    #         ...
-    #       end
-    #       class << self
-    #         include TingYun::Agent::MethodTracer
-    #         add_method_tracer :process
-    #       end
-    #     end
-    #
-    # @api public
-    module MethodTracer
-
-      def self.included klass
-        klass.extend ClassMethods
-      end
-
-      def self.extended klass
-        klass.extend ClassMethods
-      end
-
-
-      def self.trace_execution_scoped(metric_names, options={}, callback = nil) #THREAD_LOCAL_ACCESS
-        TingYun::Agent::MethodTracerHelpers.trace_execution_scoped(metric_names, options, callback) do
-          # Using an implicit block avoids object allocation for a &block param
-          yield
-        end
-      end
-
-      # Defines methods used at the class level, for adding instrumentation
-      # @api public
-      module ClassMethods
-
-        # contains methods refactored out of the #add_method_tracer method
-        module AddMethodTracer
-          # Checks to see if the method we are attempting to trace
-          # actually exists or not. #add_method_tracer can't do
-          # anything if the method doesn't exist.
-          def _method_exists?(method_name)
-            exists = method_defined?(method_name) || private_method_defined?(method_name)
-            ::TingYun::Agent.logger.error("Did not trace #{self.name}##{method_name} because that method does not exist") unless exists
-            exists
-          end
-
-          # Default to the class where the method is defined.
-          #
-          # Example:
-          #  Foo.default_metric_name_code('bar') #=> "Custom/#{Foo.name}/bar"
-          def default_metric_name_code(method_name)
-            "Custom/#{self.name}/#{method_name.to_s}"
-          end
-
-          # Checks to see if we have already traced a method with a
-          # given metric by checking to see if the traced method
-          # exists. Warns the user if methods are being double-traced
-          # to help with debugging custom instrumentation.
-          def traced_method_exists?(method_name, metric_name_code)
-            exists = method_defined?(_traced_method_name(method_name, metric_name_code))
-            ::TingYun::Agent.logger.error("Attempt to trace a method twice with the same metric: Method = #{method_name}, Metric Name = #{metric_name_code}") if exists
-            exists
-          end
-
-          # Decides which code snippet we should be eval'ing in this
-          # context, based on the options.
-          def code_to_eval(method_name, metric_name_code, options)
-            options = validate_options(method_name, options)
-            if options[:push_scope]
-              method_with_push_scope(method_name, metric_name_code, options)
-            else
-              method_without_push_scope(method_name, metric_name_code, options)
-            end
-          end
-
-          # returns an eval-able string that contains the tracing code
-          # for a fully traced metric including scoping
-          def method_with_push_scope(method_name, metric_name_code, options)
-            "def #{_traced_method_name(method_name, metric_name_code)}(*args, &block)
-              #{options[:code_header]}
-              result = ::TingYun::Agent::MethodTracerHelpers.trace_execution_scoped(\"#{metric_name_code}\",
-                        :metric => #{options[:metric]}) do
-                #{_untraced_method_name(method_name, metric_name_code)}(*args, &block)
-              end
-              #{options[:code_footer]}
-              result
-            end"
-          end
-
-          # returns an eval-able string that contains the traced
-          # method code used if the agent is not creating a scope for
-          # use in scoped metrics.
-          def method_without_push_scope(method_name, metric_name_code, options)
-            "def #{_traced_method_name(method_name, metric_name_code)}(*args, &block)
-              #{assemble_code_header(method_name, metric_name_code, options)}
-              t0 = Time.now
-              begin
-                #{_untraced_method_name(method_name, metric_name_code)}(*args, &block)\n
-              ensure
-                duration = (Time.now - t0).to_f
-                ::TingYun::Agent.record_metric(\"#{metric_name_code}\", duration)
-                #{options[:code_footer]}
-              end
-            end"
-          end
-
-          # Returns a code snippet to be eval'd that skips tracing
-          # when the agent is not tracing execution. turns
-          # instrumentation into effectively one method call overhead
-          # when the agent is disabled
-          def assemble_code_header(method_name, metric_name_code, options)
-            # header = "return #{_untraced_method_name(method_name, metric_name_code)}(*args, &block) unless TingYun::Agent.tl_is_execution_traced?\n"
-            header += options[:code_header].to_s
-            header
-          end
-
-
-
-        end
-        include AddMethodTracer
-
-
-        # Add a method tracer to the specified method.
-        #
-        # By default, this will cause invocations of the traced method to be
-        # recorded in transaction traces, and in a metric named after the class
-        # and method. It will also make the method show up in transaction-level
-        # breakdown charts and tables.
-        #
-        # === Overriding the metric name
-        #
-        # +metric_name_code+ is a string that is eval'd to get the name of the
-        # metric associated with the call, so if you want to use interpolation
-        # evaluated at call time, then single quote the value like this:
-        #
-        #     add_method_tracer :foo, 'Custom/#{self.class.name}/foo'
-        #
-        # This would name the metric according to the class of the runtime
-        # intance, as opposed to the class where +foo+ is defined.
-        #
-        # If not provided, the metric name will be <tt>Custom/ClassName/method_name</tt>.
-        #
-        # @param [Symbol] method_name the name of the method to trace
-        # @param [String] metric_name_code the metric name to record calls to
-        #   the traced method under. This may be either a static string, or Ruby
-        #   code to be evaluated at call-time in order to determine the metric
-        #   name dynamically.
-        # @param [Hash] options additional options controlling how the method is
-        #   traced.
-        # @option options [Boolean] :push_scope (true) If false, the traced method will
-        #   not appear in transaction traces or breakdown charts, and it will
-        #   only be visible in custom dashboards.
-        # @option options [Boolean] :metric (true) If false, the traced method will
-        #   only appear in transaction traces, but no metrics will be recorded
-        #   for it.
-        # @option options [String] :code_header ('') Ruby code to be inserted and run
-        #   before the tracer begins timing.
-        # @option options [String] :code_footer ('') Ruby code to be inserted and run
-        #   after the tracer stops timing.
-        #
-        # @example
-        #   add_method_tracer :foo
-        #
-        #   # With a custom metric name
-        #   add_method_tracer :foo, 'Custom/#{self.class.name}/foo'
-        #
-        #   # Instrument foo only for custom dashboards (not in transaction
-        #   # traces or breakdown charts)
-        #   add_method_tracer :foo, 'Custom/foo', :push_scope => false
-        #
-        #   # Instrument foo in transaction traces only
-        #   add_method_tracer :foo, 'Custom/foo', :metric => false
-        #
-        # @api public
-        #
-        def add_method_tracer(method_name, metric_name_code=nil, options = {})
-          return unless _method_exists?(method_name)
-          metric_name_code ||= default_metric_name_code(method_name)
-          return if traced_method_exists?(method_name, metric_name_code)
-
-          traced_method = code_to_eval(method_name, metric_name_code, options)
-
-          visibility = TingYun::Support::Helper.instance_method_visibility self, method_name
-
-          class_eval traced_method, __FILE__, __LINE__
-          alias_method _untraced_method_name(method_name, metric_name_code), method_name
-          alias_method method_name, _traced_method_name(method_name, metric_name_code)
-          send visibility, method_name
-          send visibility, _traced_method_name(method_name, metric_name_code)
-          ::TingYun::Agent.logger.debug("Traced method: class = #{self.name},"+
-                                             "method = #{method_name}, "+
-                                             "metric = '#{metric_name_code}'")
-        end
-
-        private
-
-        # given a method and a metric, this method returns the
-        # untraced alias of the method name
-        def _untraced_method_name(method_name, metric_name)
-          "#{_sanitize_name(method_name)}_without_trace_#{_sanitize_name(metric_name)}"
-        end
-
-        # given a method and a metric, this method returns the traced
-        # alias of the method name
-        def _traced_method_name(method_name, metric_name)
-          "#{_sanitize_name(method_name)}_with_trace_#{_sanitize_name(metric_name)}"
-        end
-
-        # makes sure that method names do not contain characters that
-        # might break the interpreter, for example ! or ? characters
-        # that are not allowed in the middle of method names
-        def _sanitize_name(name)
-          name.to_s.tr_s('^a-zA-Z0-9', '_')
-        end
-      end
-
-
-    end
-
-
-  end
-end
-