pgtk 0.25.0 → 0.26.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b21dae64f611c7570502547af5c940397cbc247e45f7426f5e349c6308f7fa57
-  data.tar.gz: ec130297606a929b7774b350ae74ac0a7e4b66222eeaeaaa5ba7a4e363cd11ae
+  metadata.gz: 5b7cf162ce55193a057e281c59e4ade55ba6dccee8e2076e7ebf6cc5ebd8b3d6
+  data.tar.gz: f84661c917982464a21be179bccd34e7ae0b8d3f234562dcf026e3f5aebe4793
 SHA512:
-  metadata.gz: 20a6203a23f64e00f9e9bfe9daf44463530820959f70d512d560e3dc0b73e8d9b2b2d453361d37c23a484208630d06654d139518f3188aec0eab18cb4c1817a5
-  data.tar.gz: a985cd2ab24bfc68927b46f1852ac1b06f10ad642aac41d23f8294ef66065020796f702f58df8e5c1ae03da8fc1a716a9fcc1c5349a58ee1d983e67b56398374
+  metadata.gz: 855c4d94c5ff429abe22d4593076f459df95f9cf3612157c149e929243f79e2d49bf9d56183d2c0da074e8abb30e73bff12722dbd1e57e04c3385abd8ca9314c
+  data.tar.gz: 7d9f3cfc61dfda380a6bdb63f3451ba8f09cf798796a8786fe6a0ce251e25174faf1cf21d1e4619cfed6bea95041a1072f9c62c9a60e3c825a229795d115cd6b

data/README.md

@@ -228,6 +228,9 @@ stash = Pgtk::Stash.new(
   cap: 10_000,          # Maximum cached query results (default: 10,000)
   cap_interval: 60,     # Seconds between cache size enforcement (default: 60)
   refill_interval: 16,  # Seconds between stale query refilling (default: 16)
+  refill_delay: 0.5,    # Seconds to wait before refilling the cache (default: 0)
+  retire: 60,           # Maximum age in seconds to keep a query in cache (default: 15 min)
+  retire_interval: 5.5, # How often to retire (default: 60)
   threads: 4,           # Worker threads for background refilling (default: 4)
   max_queue_length: 128 # Maximum refilling tasks in queue (default: 128)
 )

data/lib/pgtk/stash.rb

@@ -41,6 +41,8 @@ class Pgtk::Stash
 
   # Initialize a new Stash with query caching.
   #
+  # Set any of the intervals to nil to disable the cron.
+  #
   # @param [Object] pool The underlying connection pool that executes actual database queries
   # @param [Hash] stash Internal cache structure containing queries and tables hashes for sharing state
   #   across transactions
@@ -61,11 +63,11 @@ class Pgtk::Stash
   # @param [Loog] loog Logger instance for debugging and monitoring cache operations (default: null logger)
   # @param [Concurrent::ReentrantReadWriteLock] entrance Read-write lock for thread-safe cache access
   #   shared across instances
-  # @param [Concurrent::AtomicBoolean] launched Atomic boolean flag tracking whether background tasks have
-  #   been started to prevent multiple launches
   def initialize(
     pool,
     stash: { queries: {}, tables: {} },
+    loog: Loog::NULL,
+    entrance: Concurrent::ReentrantReadWriteLock.new,
     refill_interval: 16,
     refill_delay: 0,
     max_queue_length: 128,
@@ -73,14 +75,11 @@ class Pgtk::Stash
     cap: 10_000,
     cap_interval: 60,
     retire: 15 * 60,
-    retire_interval: 60,
-    loog: Loog::NULL,
-    entrance: Concurrent::ReentrantReadWriteLock.new,
-    launched: Concurrent::AtomicBoolean.new(false)
+    retire_interval: 60
   )
     @pool = pool
     @stash = stash
-    @launched = launched
+    @loog = loog
     @entrance = entrance
     @refill_interval = refill_interval
     @refill_delay = refill_delay
@@ -90,8 +89,6 @@ class Pgtk::Stash
     @cap_interval = cap_interval
     @retire = retire
     @retire_interval = retire_interval
-    @loog = loog
-    @tpool = Concurrent::FixedThreadPool.new(@threads)
   end
 
   # Start the connection pool and launch background cache management tasks.
@@ -102,8 +99,8 @@ class Pgtk::Stash
   #
   # @return [void]
   def start!
-    launch!
     @pool.start!
+    launch!
   end
 
   # Get the PostgreSQL server version.
@@ -135,19 +132,41 @@ class Pgtk::Stash
       [
         'Pgtk::Stash (',
         [
-          "refill_interval=#{@refill_interval}s",
-          "max_queue_length=#{@max_queue_length}",
           "threads=#{@threads}",
-          "cap=#{@cap}",
-          "cap_interval=#{@cap_interval}s",
-          "retire=#{@retire}",
-          "retire_interval=#{@retire_interval}s"
-        ].join(', '),
+          "max_queue_length=#{@max_queue_length}",
+          if @refill_interval
+            [
+              "refill_interval=#{@refill_interval}s",
+              "refill_delay=#{@refill_delay}s"
+            ]
+          else
+            'no refilling'
+          end,
+          if @cap_interval
+            [
+              "cap_interval=#{@cap_interval}s",
+              "cap=#{@cap}"
+            ]
+          else
+            'no capping'
+          end,
+          if @retire_interval
+            [
+              "retire_interval=#{@retire_interval}s",
+              "retire=#{@retire}"
+            ]
+          else
+            'no retirement'
+          end
+        ].flatten.join(', '),
         '):'
       ].join,
-      "  #{'not ' if @launched.false?}launched",
+      if @tpool
+        "  #{@tpool.queue_length} tasks in the thread pool"
+      else
+        '  Not launched yet'
+      end,
       "  #{stash_size} queries cached (#{stash_size > @cap ? 'above' : 'below'} the cap)",
-      "  #{@tpool.queue_length} tasks in the thread pool",
       "  #{@stash[:tables].count} tables in cache",
       "  #{qq.sum { |a| a[:s] }} stale queries in cache:",
       qq.select { |a| a[:s].positive? }.sort_by { -_1[:p] }.take(8).map do |a|
@@ -225,12 +244,8 @@ class Pgtk::Stash
       yield Pgtk::Stash.new(
         t,
         stash: @stash,
-        refill_interval: @refill_interval,
-        max_queue_length: @max_queue_length,
-        threads: @threads,
         loog: @loog,
-        entrance: @entrance,
-        launched: @launched
+        entrance: @entrance
       )
     end
   end
@@ -255,28 +270,33 @@ class Pgtk::Stash
   # @return [nil]
   # @raise [RuntimeError] if background tasks have already been launched on this cache instance
   def launch!
-    raise 'Cannot launch multiple times on same cache data' unless @launched.make_true
-    Concurrent::TimerTask.execute(execution_interval: @cap_interval, executor: @tpool) do
-      loop do
-        break if stash_size <= @cap
-        @entrance.with_write_lock do
-          @stash[:queries].each_key do |q|
-            m = @stash[:queries][q].values.map { |h| h[:used] }.min
-            next unless m
-            @stash[:queries][q].delete_if { |_, h| h[:used] == m }
-            @stash[:queries].delete_if { |_, kk| kk.empty? }
+    @tpool = Concurrent::FixedThreadPool.new(@threads)
+    if @cap_interval
+      Concurrent::TimerTask.execute(execution_interval: @cap_interval, executor: @tpool) do
+        loop do
+          break if stash_size <= @cap
+          @entrance.with_write_lock do
+            @stash[:queries].each_key do |q|
+              m = @stash[:queries][q].values.map { |h| h[:used] }.min
+              next unless m
+              @stash[:queries][q].delete_if { |_, h| h[:used] == m }
+              @stash[:queries].delete_if { |_, kk| kk.empty? }
+            end
           end
         end
       end
     end
-    Concurrent::TimerTask.execute(execution_interval: @retire_interval, executor: @tpool) do
-      @entrance.with_write_lock do
-        @stash[:queries].each_key do |q|
-          @stash[:queries][q].delete_if { |_, h| h[:used] < Time.now - @retire }
-          @stash[:queries].delete_if { |_, kk| kk.empty? }
+    if @retire_interval
+      Concurrent::TimerTask.execute(execution_interval: @retire_interval, executor: @tpool) do
+        @entrance.with_write_lock do
+          @stash[:queries].each_key do |q|
+            @stash[:queries][q].delete_if { |_, h| h[:used] < Time.now - @retire }
+            @stash[:queries].delete_if { |_, kk| kk.empty? }
+          end
         end
       end
     end
+    return unless @refill_interval
     Concurrent::TimerTask.execute(execution_interval: @refill_interval, executor: @tpool) do
       @stash[:queries]
         .map { |k, v| [k, v.values.sum { |vv| vv[:popularity] }, v.values.any? { |vv| vv[:stale] }] }
@@ -300,6 +320,5 @@ class Pgtk::Stash
         end
       end
     end
-    nil
   end
 end

data/lib/pgtk/version.rb

@@ -11,5 +11,5 @@ require_relative '../pgtk'
 # License:: MIT
 module Pgtk
   # Current version of the library.
-  VERSION = '0.25.0'
+  VERSION = '0.26.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pgtk
 version: !ruby/object:Gem::Version
-  version: 0.25.0
+  version: 0.26.0
 platform: ruby
 authors:
 - Yegor Bugayenko