erkki-production_log_analyzer 2009022401 → 2009022402

data/Rakefile

@@ -3,7 +3,7 @@ require 'hoe'
 $:.unshift './lib'
 require 'production_log/analyzer'
 
-Hoe.new 'production_log_analyzer', '2009022401' do |p|
+Hoe.new 'production_log_analyzer', '2009022402' do |p|
   p.summary = p.paragraphs_of('README.txt', 1).join ' '
   p.description = p.paragraphs_of('README.txt', 7).join ' '
   p.author = 'Eric Hodel'

data/lib/production_log/analyzer.rb

@@ -136,8 +136,6 @@ class Analyzer
   # An Array of all the request render times for the log file.
 
   attr_reader :render_times
-  
-  attr_reader :row_counts, :query_counts
 
   ##
   # Generates and sends an email report with lots of fun stuff in it.  This
@@ -177,8 +175,6 @@ class Analyzer
     @logfile_name  = logfile_name
     @request_times = Hash.new { |h,k| h[k] = [] }
     @db_times      = Hash.new { |h,k| h[k] = [] }
-    @row_counts    = Hash.new { |h,k| h[k] = [] }
-    @query_counts  = Hash.new { |h,k| h[k] = [] }
     @render_times  = Hash.new { |h,k| h[k] = [] }
   end
 
@@ -192,8 +188,6 @@ class Analyzer
         next if entry_page.nil?
         @request_times[entry_page] << entry.request_time
         @db_times[entry_page] << entry.db_time
-        @row_counts[entry_page] << entry.row_count
-        @query_counts[entry_page] << entry.query_count
         @render_times[entry_page] << entry.render_time
       end
     end
@@ -330,12 +324,12 @@ class Analyzer
 
     # header
     record = [pad_request_name("#{title} Summary"), 'Count', 'Avg', 'Std Dev',
-              'Min', 'Max', 'Queries', 'Rows']
+              'Min', 'Max']
     list << record.join("\t")
 
     # all requests
     times = records.values.flatten
-    record = [times.average, times.standard_deviation, times.min, times.max, 0, 0]
+    record = [times.average, times.standard_deviation, times.min, times.max]
     record.map! { |v| "%0.3f" % v }
     record.unshift [pad_request_name('ALL REQUESTS'), times.size]
     list << record.join("\t")
@@ -344,11 +338,7 @@ class Analyzer
     list << nil
 
     records.sort_by { |k,v| v.size}.reverse_each do |req, times|
-      if "DB Times" == title
-        average_rows = @row_counts[req].average
-        average_queries = @query_counts[req].average
-      end
-      record = [times.average, times.standard_deviation, times.min, times.max, average_queries || 0, average_rows || 0]
+      record = [times.average, times.standard_deviation, times.min, times.max]
       record.map! { |v| "%0.3f" % v }
       record.unshift ["#{pad_request_name req}", times.size]
       list << record.join("\t")

data/lib/production_log/parser.rb

@@ -105,7 +105,7 @@ module LogParser
           @request_size = log_info['Request Size'].to_i
           @response_size = log_info['Response Size'].to_i
 
-          @page = log_info['Processed'] || 'unknown'
+          @page = log_info['Processed'] if log_info['Processed']
           @page += ".#{log_info['Response Format']}" if log_info['Response Format']
 
           @db_time = log_info['DB'].split(' ').first.to_f if log_info['DB']

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: erkki-production_log_analyzer
 version: !ruby/object:Gem::Version 
-  version: "2009022401"
+  version: "2009022402"
 platform: ruby
 authors: 
 - Eric Hodel