mirror of
https://github.com/discourse/discourse.git
synced 2024-12-12 06:43:41 +08:00
d7b48d0229
Themes often cache `nil` values in a DistributedCache. This bug meant that we were re-calculating some values on every request, AND triggering message-bus publishing on every request. This fix should provide a significant performance improvement for busy sites.
30 lines
707 B
Ruby
30 lines
707 B
Ruby
# frozen_string_literal: true
|
|
|
|
require 'message_bus/distributed_cache'
|
|
|
|
class DistributedCache < MessageBus::DistributedCache
|
|
def initialize(key, manager: nil, namespace: true)
|
|
super(
|
|
key,
|
|
manager: manager,
|
|
namespace: namespace,
|
|
app_version: Discourse.git_version
|
|
)
|
|
end
|
|
|
|
# Defer setting of the key in the cache for performance critical path to avoid
|
|
# waiting on MessageBus to publish the message which involves writing to Redis.
|
|
def defer_set(k, v)
|
|
Scheduler::Defer.later("#{@key}_set") do
|
|
self[k] = v
|
|
end
|
|
end
|
|
|
|
def defer_get_set(k, &block)
|
|
return self[k] if hash.key? k
|
|
value = block.call
|
|
self.defer_set(k, value)
|
|
value
|
|
end
|
|
end
|