lookout-query_reviewer 0.1.5

data/lib/query_reviewer/views/_query_with_warning.html.erb

@@ -0,0 +1,54 @@
+<li id="query_<%= query_with_warning.id %>">
+	<div>
+		<%= render :partial => "/spectrum", :locals => {:severity => query_with_warning.max_severity} %>
+    <% if QueryReviewer::CONFIGURATION["production_data"] %>
+      <div style="float: left; padding-right: 5px;">
+        <%= duration_with_color(query_with_warning) %>s
+      </div>
+    <% end %>
+    <p>
+      <% if query_with_warning.count > 1 %>
+        <b title="<%= query_with_warning.count %> queries were executed with the same stack trace and similar SQL structure">
+            <%= query_with_warning.count  %> identical queries
+        </b>
+      <% end %>
+      <i>Table <%= (query_with_warning.warnings.detect {|w| !w.table.blank? } || query_with_warning.warnings.last).table %>:</i>
+			<% query_with_warning.warnings.sort{|a,b| a.severity <=> b.severity}.reverse.each_with_index do |warn, index| %>
+				<span style="color: <%= severity_color warn.severity%>;" title="<%= warn.desc%>"><%= warn.problem %></span><%= ", " if index < query_with_warning.warnings.length - 1 %>
+			<% end %>
+			<a href="#" onclick="query_review_toggle('warning_<%= query_with_warning.id %>_desc')" title="show/hide warning message">MSG</a>
+			<a href="#" onclick="query_review_toggle('warning_<%= query_with_warning.id %>_sql')" title="show/hide sql">SQL</a>
+      <% if query_with_warning.select? %>
+      	<a href="#" onclick="query_review_toggle('warning_<%= query_with_warning.id %>_explain')" title="show/hide explain output">EXPLN</a>
+      <% end %>
+      <% if QueryReviewer::CONFIGURATION["profiling"] && query_with_warning.profile %>
+  			<a href="#" onclick="query_review_toggle('warning_<%= query_with_warning.id %>_profile')" title="show/hide profile output">PROF</a>
+      <% end %>
+      <a href="#" onclick="query_review_toggle('warning_<%= query_with_warning.id %>_trace')" title="show/hide stack trace">TRACE</a>
+			<% if ignore_hash?(query_with_warning.to_hash) %>
+				<a href="#" onclick="remove_ignore_hash('<%= query_with_warning.to_hash %>'); query_review_hide('query_<%= query_with_warning.id %>')" title="stop ignore this query from now on">UNIGNR</a>
+			<% else %>
+				<a href="#" onclick="add_ignore_hash('<%= query_with_warning.to_hash %>'); query_review_hide('query_<%= query_with_warning.id %>')" title="ignoring this query from now on">IGNR</a>
+			<% end %>
+		</p>
+	</div>
+	<p style="display: none" id="warning_<%= query_with_warning.id %>_desc" class="indent">
+		<% query_with_warning.warnings.each do |warn| %>
+			<span style="color: <%= severity_color warn.severity%>"><%= warn.desc %></span><br/>
+		<% end %>
+	</p>
+	<p style="display: none" id="warning_<%= query_with_warning.id %>_sql" class="indent small tbpadded">
+		<%= render :partial => "/query_sql", :locals => {:query_sql => query_with_warning} %>
+	</p>
+	<div style="display: none" id="warning_<%= query_with_warning.id %>_explain" class="indent small tbpadded">
+		<%= render :partial => "/explain", :locals => {:query => query_with_warning} %>
+	</div>
+  <% if QueryReviewer::CONFIGURATION["profiling"] && query_with_warning.profile %>
+    <div style="display: none" id="warning_<%= query_with_warning.id %>_profile" class="indent small tbpadded">
+      <%= render :partial => "/profile", :locals => {:query => query_with_warning} %>
+    </div>
+  <% end %>
+  <div style="display: none" id="warning_<%= query_with_warning.id %>_trace" class="indent small">
+		<%= render :partial => "/query_trace", :locals => {:query_id => query_with_warning.id, :full_trace => query_with_warning.full_trace,:query_trace => query_with_warning.relevant_trace} %>
+	</div>
+</li>

data/lib/query_reviewer/views/_spectrum.html.erb

@@ -0,0 +1,10 @@
+<div class="spectrum_container" title="Severity: <%= severity%> out of 10">
+	<% 0.upto(15) do |i| %>
+		<div class="spectrum_elem" style="background-color: <%= "##{i.to_s(16)}F0"%>"></div>
+	<% end %>
+	<% 0.upto(15) do |i| %>
+		<div class="spectrum_elem" style="background-color: <%= "#F#{((15-i)).to_s(16)}0"%>"></div>
+	<% end %>
+	<div class="spectrum_elem" style="background-color: #FF0000"></div>
+	<div class="spectrum_pointer" style="left: <%= severity * 3 + 2 %>px;"></div>
+</div>

data/lib/query_reviewer/views/_warning_no_query.html.erb

@@ -0,0 +1,8 @@
+<li id="warning_<%= warning_no_query.id %>">
+	<div>
+		<%= render :partial => "/spectrum", :locals => {:severity => warning_no_query.severity} %>
+		<p>
+			<span style="color: <%= severity_color warning_no_query.severity%>" title="<%= warning_no_query.desc %>"><%= warning_no_query.problem %></span>
+		</p>
+	</div>
+</li>

data/lib/query_reviewer/views/query_review_box_helper.rb

@@ -0,0 +1,99 @@
+module QueryReviewer
+  module Views
+    module QueryReviewBoxHelper
+      def parent_div_class
+        "sql_#{parent_div_status.downcase}"
+      end
+
+      def parent_div_status
+        if !enabled_by_cookie
+          "DISABLED"
+        elsif overall_max_severity < (QueryReviewer::CONFIGURATION["warn_severity"] || 4)
+          "OK"
+        elsif overall_max_severity < (QueryReviewer::CONFIGURATION["critical_severity"] || 7)
+          # uh oh
+          "WARNING"
+        else
+          # oh @#&!
+          "CRITICAL"
+        end
+      end
+
+      def syntax_highlighted_sql(sql)
+        sql = sql.to_sql if sql.respond_to?(:to_sql)
+        if QueryReviewer::CONFIGURATION["uv"]
+          uv_out = Uv.parse(sql, "xhtml", "sql_rails", false, "blackboard")
+          uv_out.gsub("<pre class=\"blackboard\">", "<code class=\"sql\">").gsub("</pre>", "</code>")
+        else
+          sql.gsub(/</, "&lt;").gsub(/>/, "&gt;")
+        end
+      end
+
+      def overall_max_severity
+        max = 0
+        max = queries_with_warnings_sorted_nonignored[0].max_severity unless queries_with_warnings_sorted_nonignored.empty?
+        max = warnings_no_query_sorted.first.severity unless warnings_no_query_sorted.empty? || warnings_no_query_sorted.first.severity < max
+        max
+      end
+
+      def severity_color(severity)
+        red = (severity * 16.0 / 10).to_i
+        green = ((10-severity) * 16.0 / 10).to_i
+        red = 8 if red > 8
+        red = 0 if red < 0
+        green = 8 if green > 8
+        green = 0 if green < 0
+        "##{red.to_s(16)}#{green.to_s(16)}0"
+      end
+
+      def ignore_hash?(h)
+        (controller.send(:cookies)["query_review_ignore_list"] || "").split(",").include?(h.to_s)
+      end
+
+      def queries_with_warnings
+        @queries.queries.select{|q| q.has_warnings?}
+      end
+
+      def queries_with_warnings_sorted
+        queries_with_warnings.sort{|a,b| (b.max_severity * 1000 + (b.duration || 0)) <=> (a.max_severity * 1000 + (a.duration || 0))}
+      end
+
+      def queries_with_warnings_sorted_nonignored
+        queries_with_warnings_sorted.select{|q| q.max_severity >= ::QueryReviewer::CONFIGURATION["warn_severity"] && !ignore_hash?(q.to_hash)}
+      end
+
+      def queries_with_warnings_sorted_ignored
+        queries_with_warnings_sorted.reject{|q| q.max_severity >= ::QueryReviewer::CONFIGURATION["warn_severity"] && !ignore_hash?(q.to_hash)}
+      end
+
+      def warnings_no_query_sorted
+        @queries.collection_warnings.sort{|a,b| a.severity <=> b.severity}.reverse
+      end
+
+      def warnings_no_query_sorted_ignored
+        warnings_no_query_sorted.select{|q| q.severity < ::QueryReviewer::CONFIGURATION["warn_severity"]}
+      end
+
+      def warnings_no_query_sorted_nonignored
+        warnings_no_query_sorted.select{|q| q.severity >= ::QueryReviewer::CONFIGURATION["warn_severity"]}
+      end
+
+      def enabled_by_cookie
+        controller.send(:cookies)["query_review_enabled"]
+      end
+
+      def duration_with_color(query)
+        title = query.duration_stats
+        duration = query.duration
+        span_html = if duration > QueryReviewer::CONFIGURATION["critical_duration_threshold"]
+          "<span style=\"color: #{severity_color(9)}\" title=\"#{title}\">#{"%.3f" % duration}</span>"
+        elsif duration > QueryReviewer::CONFIGURATION["warn_duration_threshold"]
+          "<span style=\"color: #{severity_color(QueryReviewer::CONFIGURATION["critical_severity"])}\" title=\"#{title}\">#{"%.3f" % duration}</span>"
+        else
+          "<span title=\"#{title}\">#{"%.3f" % duration}</span>"
+        end
+        span_html.respond_to?(:html_safe) ? span_html.html_safe : span_html
+      end
+    end
+  end
+end

data/query_reviewer_defaults.yml

@@ -0,0 +1,39 @@
+all:
+    inject_view: true
+    stack_trace_lines: 3
+    trace_includes_vendor: false
+    trace_includes_lib: true
+    profiling: enabled
+    production_data: true
+    max_safe_key_length: 100
+    disable_sql_cache: true    
+
+    warn_severity: 3
+    critical_severity: 7
+
+    warn_select_count: 20
+    critical_select_count: 50
+
+    warn_update_count: 5
+    critical_update_count: 10
+
+    warn_insert_count: 5
+    critical_insert_count: 10
+
+    warn_delete_count: 5
+    critical_delete_count: 10
+
+    warn_duration_threshold: 0.2
+    critical_duration_threshold: 1.0
+
+    warn_affected_rows: 10
+    critical_affected_rows: 100
+
+development:
+    enabled: true
+
+production:
+    enabled: false
+
+test:
+    enabled: false

metadata

@@ -0,0 +1,77 @@
+--- !ruby/object:Gem::Specification
+name: lookout-query_reviewer
+version: !ruby/object:Gem::Version
+  version: 0.1.5
+  prerelease: 
+platform: ruby
+authors:
+- dsboulder, nesquena
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2012-08-17 00:00:00.000000000Z
+dependencies: []
+description: Runs explain before each select query and displays results in an overlayed
+  div
+email: nesquena@gmail.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- MIT-LICENSE
+- Rakefile
+- README.md
+- query_reviewer_defaults.yml
+- lib/query_reviewer.rb
+- lib/query_reviewer/rails.rb
+- lib/query_reviewer/mysql_analyzer.rb
+- lib/query_reviewer/sql_query.rb
+- lib/query_reviewer/controller_extensions.rb
+- lib/query_reviewer/query_warning.rb
+- lib/query_reviewer/sql_query_collection.rb
+- lib/query_reviewer/views/_box_header.html.erb
+- lib/query_reviewer/views/_js_includes_new.html.erb
+- lib/query_reviewer/views/_js_includes.html.erb
+- lib/query_reviewer/views/_box_ajax.js
+- lib/query_reviewer/views/_query_trace.html.erb
+- lib/query_reviewer/views/query_review_box_helper.rb
+- lib/query_reviewer/views/_profile.html.erb
+- lib/query_reviewer/views/_query_sql.html.erb
+- lib/query_reviewer/views/_box.html.erb
+- lib/query_reviewer/views/_warning_no_query.html.erb
+- lib/query_reviewer/views/_box_body.html.erb
+- lib/query_reviewer/views/_query_with_warning.html.erb
+- lib/query_reviewer/views/_spectrum.html.erb
+- lib/query_reviewer/views/_box_includes.html.erb
+- lib/query_reviewer/views/_box_disabled.html.erb
+- lib/query_reviewer/views/_explain.html.erb
+- lib/query_reviewer/tasks.rb
+- lib/query_reviewer/mysql_adapter_extensions.rb
+- lib/query_reviewer/sql_sub_query.rb
+- lib/query_reviewer/array_extensions.rb
+homepage: https://github.com/lookout/query_reviewer
+licenses: []
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 1.8.10
+signing_key: 
+specification_version: 3
+summary: Runs explain before each select query and displays results in an overlayed
+  div
+test_files: []