rack-mini-profiler 0.1.15.pre → 0.1.20

data/CHANGELOG

@@ -69,3 +69,25 @@
   * 1.15.pre
   * fixed annoying bug where client settings were not sticking 
   * fixed long standing issue with Rack::ConditionalGet stopping MiniProfiler from working properly
+
+5-September-2012 - Sam 
+
+  * 1.16
+  * fixed long standing problem specs (issue with memory store)
+  * fixed issue where profiler would be dumped when you got a 404 in production (and any time rails is bypassed)
+  * implemented stacktrace properly
+
+9-September-2012 - Sam
+
+  * 1.17
+  * pp=sample was bust unless stacktrace was installed
+
+10-September-2012 - Sam 
+
+  * 1.19
+  * fix compat issue with 1.8.7
+
+12-September-2012 - Sam
+  
+  * 1.20
+  * Added pp=profile-gc , it allows you to profile the GC in Ruby 1.9.3

data/lib/html/includes.tmpl

@@ -138,7 +138,7 @@
     </td>
 
   {{if HasSqlTimings}}
-    <td class="profiler-duration {{if HasDuplicateSqlTimings}}profiler-warning{{/if}}" title="{{if HasDuplicateSqlTimings}}duplicate queries detected - {{/if}}${ExecutedReaders} reader, ${ExecutedScalars} scalar, ${ExecutedNonQueries} non-query statements executed">
+    <td class="profiler-duration {{if HasDuplicateSqlTimings}}profiler-warning{{/if}}" title="{{if HasDuplicateSqlTimings}}duplicate queries detected - {{/if}}{{if ExecutedReaders > 0 || ExecutedScalars > 0 || ExecutedNonQueries > 0}}${ExecutedReaders} reader, ${ExecutedScalars} scalar, ${ExecutedNonQueries} non-query statements executed{{/if}}">
       <a class="profiler-queries-show">
         {{if HasDuplicateSqlTimings}}<span class="profiler-nuclear">!</span>{{/if}}
         ${SqlTimings.length} <span class="profiler-unit">sql</span>

data/lib/mini_profiler/profiler.rb

@@ -19,7 +19,7 @@ module Rack
 
 	class MiniProfiler
 
-		VERSION = '106'.freeze
+		VERSION = '107'.freeze
 
     class << self 
       
@@ -167,6 +167,7 @@ module Rack
 
 
 		def call(env)
+
       client_settings = ClientSettings.new(env)
 
       status = headers = body = nil
@@ -207,6 +208,10 @@ module Rack
         return [status,headers,body]
       end
 
+      if query_string =~ /pp=profile-gc/
+        return profile_gc(env)
+      end
+
       MiniProfiler.create_current(env, @config)
       MiniProfiler.deauthorize_request if @config.authorization_mode == :whitelist
       if query_string =~ /pp=normal-backtrace/
@@ -224,17 +229,30 @@ module Rack
       done_sampling = false
       quit_sampler = false
       backtraces = nil
-      missing_stacktrace = false
+      stacktrace_installed = true
       if query_string =~ /pp=sample/
+        skip_frames = 0
         backtraces = []
         t = Thread.current
+        
+        begin 
+          require 'stacktrace'
+          skip_frames = stacktrace.length
+        rescue LoadError
+          stacktrace_installed = false
+        end
+
         Thread.new {
           begin
             i = 10000 # for sanity never grab more than 10k samples 
             while i > 0
               break if done_sampling
               i -= 1
-              backtraces << t.backtrace
+              if stacktrace_installed
+                backtraces << t.stacktrace(0,-(1+skip_frames), StackFrame::Flags::METHOD | StackFrame::Flags::KLASS)
+              else
+                backtraces << t.backtrace 
+              end
               sleep 0.001
             end
           ensure
@@ -263,10 +281,14 @@ module Rack
 
       skip_it = current.discard
       if (config.authorization_mode == :whitelist && !MiniProfiler.request_authorized?)
-        client_settings.discard_cookie!(headers)
+        # this is non-obvious, don't kill the profiling cookie on errors or short requests
+        # this ensures that stuff that never reaches the rails stack does not kill profiling
+        if status == 200 && ((Time.now - start) > 0.1) 
+          client_settings.discard_cookie!(headers)
+        end
         skip_it = true
       end
-      
+
       return [status,headers,body] if skip_it
 
       # we must do this here, otherwise current[:discard] is not being properly treated
@@ -277,7 +299,7 @@ module Rack
 
       if query_string =~ /pp=help/
         body.close if body.respond_to? :close
-        return help(nil, client_settings)
+        return help(client_settings)
       end
       
       page_struct = current.page_struct
@@ -285,7 +307,6 @@ module Rack
 
       if backtraces
         body.close if body.respond_to? :close
-        return help(:stacktrace, client_settings) if missing_stacktrace
         return analyze(backtraces, page_struct)
       end
       
@@ -355,7 +376,7 @@ module Rack
       [200, headers, [body]]
     end
 
-    def help(category = nil, client_settings)
+    def help(client_settings)
       headers = {'Content-Type' => 'text/plain'}
       body = "Append the following to your query string:
 
@@ -365,18 +386,73 @@ module Rack
   pp=no-backtrace #{"(*) " if client_settings.backtrace_none?}: don't collect stack traces from all the SQL executed (sticky, use pp=normal-backtrace to enable)
   pp=normal-backtrace #{"(*) " if client_settings.backtrace_default?}: collect stack traces from all the SQL executed and filter normally
   pp=full-backtrace #{"(*) " if client_settings.backtrace_full?}: enable full backtraces for SQL executed (use pp=normal-backtrace to disable) 
-  pp=sample : sample stack traces and return a report isolating heavy usage (experimental)
+  pp=sample : sample stack traces and return a report isolating heavy usage (experimental works best with the stacktrace gem)
   pp=disable : disable profiling for this session 
   pp=enable : enable profiling for this session (if previously disabled)
+  pp=profile-gc: perform gc profiling on this request (ruby 1.9.3 only)
 "
-      if (category == :stacktrace)
-        body = "pp=stacktrace requires the stacktrace gem - add gem 'stacktrace' to your Gemfile"
-      end
     
       client_settings.write!(headers)
       [200, headers, [body]]
     end
 
+
+    def object_space_stats
+      stats = {}
+      ObjectSpace.each_object { |o|
+        stats[o.class] ||= 1
+        stats[o.class] += 1
+      }
+      stats
+    end
+
+    def diff_object_stats(before,after)
+      diff = {}
+      after.each do |k,v|
+        diff[k] = v - (before[k] || 0)
+      end
+      before.each do |k,v|
+        diff[k] = 0 - v unless after[k]
+      end
+
+      diff
+    end
+
+    def profile_gc(env)
+      
+      body = "";
+
+      stat_before = object_space_stats
+      begin
+        GC::Profiler.clear
+        GC::Profiler.enable
+        @app.call(env)
+        body << GC::Profiler.result
+      ensure
+        GC::Profiler.disable
+      end
+      stat_after = object_space_stats
+
+      diff = diff_object_stats(stat_before,stat_after)
+
+      body << "
+ObjectSpace delta caused by request:
+--------------------------------------------\n"
+      diff.to_a.reject{|k,v| v == 0}.sort{|x,y| y[1] <=> x[1]}.each do |k,v|
+        body << "#{k} : #{v}\n" if v != 0
+      end
+
+      body << "\n
+ObjectSpace stats:
+-----------------\n"
+
+      stat_after.to_a.sort{|x,y| y[1] <=> x[1]}.each do |k,v|
+        body << "#{k} : #{v}\n" 
+      end
+
+      return [200, {'Content-Type' => 'text/plain'}, body]
+    end
+
     def analyze(traces, page_struct)
       headers = {'Content-Type' => 'text/plain'}
       body = "Collected: #{traces.count} stack traces. Duration(ms): #{page_struct.duration_ms}"
@@ -387,6 +463,7 @@ module Rack
         fulldump << "\n\n"
         distinct = {}
         trace.each do |frame|
+          frame = "#{frame.klass}::#{frame.method}" unless String === frame
           unless distinct[frame]
             distinct[frame] = true
             seen[frame] ||= 0

data/lib/mini_profiler/profiling_methods.rb

@@ -8,6 +8,22 @@ module Rack
 			  c.current_timer.add_sql(query, elapsed_ms, c.page_struct, c.skip_backtrace, c.full_backtrace) if (c && c.current_timer)
 		  end
 
+      def start_step(name)
+        if current
+          parent_timer = current.current_timer
+          current.current_timer = current_timer = current.current_timer.add_child(name)
+          [current_timer,parent_timer]
+        end
+      end
+
+      def finish_step(obj)
+        if obj && current
+          current_timer, parent_timer = obj
+          current_timer.record_time
+          current.current_timer = parent_timer
+        end
+      end
+
       # perform a profiling step on given block
       def step(name, opts = nil)
         if current
@@ -26,8 +42,11 @@ module Rack
       end
 
       def unprofile_method(klass, method)
-        with_profiling = (method.to_s + "_with_mini_profiler").intern
-        without_profiling = (method.to_s + "_without_mini_profiler").intern
+
+        clean = clean_method_name(method)
+
+        with_profiling = ("#{clean}_with_mini_profiler").intern
+        without_profiling = ("#{clean}_without_mini_profiler").intern
         
         if klass.send :method_defined?, with_profiling
           klass.send :alias_method, method, without_profiling
@@ -38,8 +57,10 @@ module Rack
 
       def profile_method(klass, method, &blk)
         default_name = klass.to_s + " " + method.to_s
-        with_profiling = (method.to_s + "_with_mini_profiler").intern
-        without_profiling = (method.to_s + "_without_mini_profiler").intern
+        clean = clean_method_name(method)
+
+        with_profiling =  ("#{clean}_with_mini_profiler").intern
+        without_profiling = ("#{clean}_without_mini_profiler").intern
         
         if klass.send :method_defined?, with_profiling
           return # dont double profile
@@ -67,6 +88,13 @@ module Rack
         end
         klass.send :alias_method, method, with_profiling
       end
+
+      private
+
+      def clean_method_name(method)
+        method.to_s.gsub(/[\?\!]/, "")
+      end
+
     end
   end
 end

data/lib/mini_profiler/storage/memory_store.rb

@@ -55,7 +55,7 @@ module Rack
       def cleanup_cache
         expire_older_than = ((Time.now.to_f - MiniProfiler::MemoryStore::EXPIRE_TIMER_CACHE) * 1000).to_i
         @timer_struct_lock.synchronize {
-          @timer_struct_cache.delete_if { |k, v| v['Root']['StartMilliseconds'] < expire_older_than }
+          @timer_struct_cache.delete_if { |k, v| v['Started'] < expire_older_than }
         }
       end
     end

data/rack-mini-profiler.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
 	s.name = "rack-mini-profiler"
-	s.version = "0.1.15.pre"
+	s.version = "0.1.20"
 	s.summary = "Profiles loading speed for rack applications."
 	s.authors = ["Aleks Totic","Sam Saffron", "Robin Ward"]
 	s.description = "Page loading speed displayed on every page. Optimize while you develop, performance is a feature."

metadata

@@ -1,8 +1,8 @@
 --- !ruby/object:Gem::Specification
 name: rack-mini-profiler
 version: !ruby/object:Gem::Version
-  version: 0.1.15.pre
-  prerelease: 7
+  version: 0.1.20
+  prerelease: 
 platform: ruby
 authors:
 - Aleks Totic
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-09-04 00:00:00.000000000 Z
+date: 2012-09-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -131,13 +131,16 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 439688077
+      hash: -250357363
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
-  - - ! '>'
+  - - ! '>='
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
+      segments:
+      - 0
+      hash: -250357363
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24