query_owl 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9dd0e38c57b1b0c94a99a0ffb22f6ecc71f0e28c6d4d6d041dac57bc6e5bf626
-  data.tar.gz: dd6c94c94314dc38da3a757d0e4147dc349ae8842f10e0e29f9fa7fc2e0727ed
+  metadata.gz: 1269c2c24be94cd4ebf8ef07d12fa3e5da1aa2c1573256779f641d3f0a089c6e
+  data.tar.gz: 891f6323d6a6ffd95b6e73e131ddbf2f556e90703f5aa67819d92a3bdf0b4ccb
 SHA512:
-  metadata.gz: be09a0350d981ffcd6e2b5b4e57825daf3d4aca7a0747d0f3579188df22ddc5f26babeab619c24e1d47c0e86ebda1dc1dce99ee65ce32cc01ff1cef1f997a95e
-  data.tar.gz: 6d499e2d7972b3252f6eccaae8e415f742e2e28b579c08d9e53f2c7377a998a6c5b7be1a46bc3f4189d353258fe7dc0dd01f5df29eb4986d7851c07ad1179fca
+  metadata.gz: 6fe7f83cbb596f8616ffdd11ccea40c4c2c485fa16b303f783a0b6b3a551eef4b8313e353585c8990061832227926738931fa582f9d2de8860b819afad301b96
+  data.tar.gz: 87244c9aba6eff9996e97d1f5a3727b8fbf283d6b10647c05fb903d4d7833bc464fdd0c26db84895f8ed7dbc7e290bc6c22f5f225f01360649d03859b9e2dfb7

data/app/assets/stylesheets/query_owl/_03_table.css

@@ -26,4 +26,9 @@
 }
 
 .qo-table tr:last-child td { border-bottom: none; }
-.qo-table tr:hover td      { background: #fafafa; }
+.qo-table tr:hover td      { background: #fafafa; }
+
+.qo-table td:nth-child(2) { max-width: 320px; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
+
+.qo-request  { white-space: nowrap; }
+.qo-path     { font-family: ui-monospace, "SFMono-Regular", Menlo, monospace; font-size: 11px; opacity: .7; }

data/app/views/query_owl/slow_queries/index.html.erb

@@ -12,6 +12,7 @@
         <th>Type</th>
         <th>SQL / Details</th>
         <th>Info</th>
+        <th>Request</th>
         <th>Recorded At</th>
         <th>Backtrace</th>
       </tr>
@@ -25,6 +26,14 @@
             <% if event[:count] %>count: <%= event[:count] %><% end %>
             <% if event[:duration_ms] %><%= event[:duration_ms] %>ms<% end %>
           </td>
+          <td class="qo-request qo-muted">
+            <% if event[:controller] || event[:action] %>
+              <span class="qo-monospace"><%= [event[:controller], event[:action]].compact.join("#") %></span>
+            <% end %>
+            <% if event[:path] %>
+              <br><span class="qo-path"><%= event[:path] %></span>
+            <% end %>
+          </td>
           <td class="qo-muted"><%= event[:recorded_at]&.strftime("%H:%M:%S") %></td>
           <td class="qo-monospace qo-muted"><%= Array(event[:backtrace]).first %></td>
         </tr>

data/lib/query_owl/configuration.rb

@@ -12,6 +12,11 @@ module QueryOwl
     end
 
     def notifiers=(arr)
+      arr.each do |notifier|
+        unless notifier.respond_to?(:call)
+          raise ArgumentError, "notifiers must respond to #call (#{notifier.class} does not)"
+        end
+      end
       @notifiers = arr
     end
 

data/lib/query_owl/event_store.rb

@@ -4,7 +4,7 @@ module QueryOwl
       def push(event)
         mutex.synchronize do
           ensure_buffer_size
-          buffer[@write_pos] = event.merge(recorded_at: Time.now)
+          buffer[@write_pos] = event.merge(recorded_at: Time.current)
           @write_pos = (@write_pos + 1) % capacity
           @stored    = [@stored + 1, capacity].min
         end

data/lib/query_owl/file_logger.rb

@@ -1,4 +1,5 @@
 require "json"
+require "fileutils"
 
 module QueryOwl
   class FileLogger
@@ -9,9 +10,12 @@ module QueryOwl
         path = QueryOwl.config.log_file
         return unless path
 
+        FileUtils.mkdir_p(File.dirname(path))
         File.open(path, "a") do |f|
           events.each { |e| f.puts(JSON.generate(serializable(e))) }
         end
+      rescue => e
+        Rails.logger.error "[QueryOwl] FileLogger failed: #{e.message}"
       end
 
       private

data/lib/query_owl/middleware.rb

@@ -28,7 +28,13 @@ module QueryOwl
                     Detector.detect_slow_queries(queries) +
                     Detector.detect_unused_eager_loads(eager_data))
                    .map { |e| e.merge(context) }
-      events.each { |event| QueryOwl.config.notifiers.each { |notifier| notifier.call(event) } }
+      events.each do |event|
+        QueryOwl.config.notifiers.each do |notifier|
+          notifier.call(event)
+        rescue => e
+          Rails.logger.error "[QueryOwl] Notifier #{notifier.class} raised: #{e.message}"
+        end
+      end
       Logger.log_summary(events)
       events.each { |e| EventStore.push(e) }
       FileLogger.append(events)

data/lib/query_owl/version.rb

@@ -1,3 +1,3 @@
 module QueryOwl
-  VERSION = "0.4.0"
+  VERSION = "0.4.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: query_owl
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Chuck Smith