Skip to content

Commit 0b805eb

Browse files
authored
Add missing caching information to logs (reidmorrison#126)
* Fix logging of cache_hit and cache_hits * Add whether a sql statement came from the query cache to ActiveRecord log statements
1 parent 07d288f commit 0b805eb

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

lib/rails_semantic_logger/action_view/log_subscriber.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ def render_partial(event)
4040
partial: from_rails_root(event.payload[:identifier])
4141
}
4242
payload[:within] = from_rails_root(event.payload[:layout]) if event.payload[:layout]
43-
payload[:cache] = payload[:cache_hit] unless event.payload[:cache_hit].nil?
43+
payload[:cache] = event.payload[:cache_hit] unless event.payload[:cache_hit].nil?
4444
payload[:allocations] = event.allocations if event.respond_to?(:allocations)
4545

4646
logger.measure(
@@ -60,7 +60,7 @@ def render_collection(event)
6060
template: from_rails_root(identifier),
6161
count: event.payload[:count]
6262
}
63-
payload[:cache_hits] = payload[:cache_hits] if payload[:cache_hits]
63+
payload[:cache_hits] = event.payload[:cache_hits] if event.payload[:cache_hits]
6464
payload[:allocations] = event.allocations if event.respond_to?(:allocations)
6565

6666
logger.measure(

lib/rails_semantic_logger/active_record/log_subscriber.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ def sql(event)
3232
log_payload = {sql: payload[:sql]}
3333
log_payload[:binds] = bind_values(payload) unless (payload[:binds] || []).empty?
3434
log_payload[:allocations] = event.allocations if event.respond_to?(:allocations)
35+
log_payload[:cached] = event.payload[:cached]
3536

3637
log = {
3738
message: name,

0 commit comments

Comments
 (0)