pgtk 0.21.3 → 0.21.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 96d8784e334eb788333eb79d26c99c47f129ea8de8e522732fa37fa333b99a35
-  data.tar.gz: ec5caad7117ec1f22784c718316b2897effb907f03cd1043e7305a057d9412ee
+  metadata.gz: 35204d3374563b14809ba4c71c020294c14f0914967e62157a8aa5b005816eb4
+  data.tar.gz: ba2972020329f362a4c6ef818cb938bff243a64ea0d19c31d5028c5c3808a07d
 SHA512:
-  metadata.gz: 2266b9797e6f49cecdd4f5e4b198f4d06d9060544a6e2eb89ca99c8f0a05a93e6674b6237af30569e6d474c08e13b02ac706cf9a96daf07938021cd61d518e51
-  data.tar.gz: 1b723ca7237b8d1e5381481599fd0850e2685fa411a37551360d78548ca0b29ea38da50b849dbad37c8cac1d65422cdcb488b1edd45d9025ccf67336ea586048
+  metadata.gz: 53afb6021dccbd51901eadb93ee5ac8dfce38d8a40f45f400e5b3f5994bc9f2a3a23e6dcb694b7bf7211a00f7195c1fda8871e22c06b56bea050c766d59d25f1
+  data.tar.gz: 4973efc117e972436f4e447c730974918159a58d2120aeb94ad620ef163ad95756deeb957549c727facdc15a75a36e5c3632f1182bdfa0779c921ec50dc0e7fe

data/lib/pgtk/stash.rb

@@ -38,28 +38,24 @@ class Pgtk::Stash
   # @param [Hash] stash Optional existing stash to use (default: new empty stash)
   # @option [Hash] queries Internal cache data (default: {})
   # @option [Hash] tables Internal cache data (default: {})
-  # @option [Concurrent::ReentrantReadWriteLock] entrance Lock for write internal state
-  # @option [Concurrent::AtomicBoolean] launched Latch for start timers once
   # @param [Integer] refill_interval Interval in seconds for recalculate stale queries
   # @param [Integer] top Number of queries to recalculate
   # @param [Integer] threads Number of threads in threadpool
   # @param [Loog] loog Logger for debugging (default: null logger)
   def initialize(
     pool,
-    stash = {
-      queries: {},
-      tables: {},
-      entrance: Concurrent::ReentrantReadWriteLock.new,
-      launched: Concurrent::AtomicBoolean.new(false)
-    },
+    stash: { queries: {}, tables: {} },
     refill_interval: 5,
     top: 100,
     threads: 5,
-    loog: Loog::NULL
+    loog: Loog::NULL,
+    entrance: Concurrent::ReentrantReadWriteLock.new,
+    launched: Concurrent::AtomicBoolean.new(false)
   )
     @pool = pool
     @stash = stash
-    @entrance = stash[:entrance]
+    @launched = launched
+    @entrance = entrance
     @refill_interval = refill_interval
     @top = top
     @threads = threads
@@ -93,7 +89,7 @@ class Pgtk::Stash
       @pool.dump,
       '',
       "Pgtk::Stash (refill_interval=#{@refill_interval}s, top=#{@top}q, threads=#{@threads}t):",
-      "  #{'not ' if @stash[:launched].false?}launched",
+      "  #{'not ' if @launched.false?}launched",
       "  #{@stash[:tables].count} tables in cache",
       "  #{@stash[:queries].count} queries in cache:",
       qq.sort_by { -_1[2] }.take(20).map { |a| "    #{a[1]}/#{a[2]}p/#{a[3]}s: #{a[0]}" }
@@ -128,9 +124,9 @@ class Pgtk::Stash
       if ret.nil? || @stash.dig(:queries, pure, key, :stale)
         ret = @pool.exec(pure, params, result)
         unless pure.include?(' NOW() ')
+          tables = pure.scan(/(?<=^|\s)(?:FROM|JOIN) ([a-z_]+)(?=\s|$)/).map(&:first).uniq
+          raise "No tables at #{pure.inspect}" if tables.empty?
           @entrance.with_write_lock do
-            tables = pure.scan(/(?<=^|\s)(?:FROM|JOIN) ([a-z_]+)(?=\s|$)/).map(&:first).uniq
-            raise "No tables at #{pure.inspect}" if tables.empty?
             tables.each do |t|
               @stash[:tables][t] = [] if @stash[:tables][t].nil?
               @stash[:tables][t].append(pure).uniq!
@@ -159,11 +155,14 @@ class Pgtk::Stash
   def transaction
     @pool.transaction do |t|
       yield Pgtk::Stash.new(
-        t, @stash,
+        t,
+        stash: @stash,
         refill_interval: @refill_interval,
         top: @top,
         threads: @threads,
-        loog: @loog
+        loog: @loog,
+        entrance: @entrance,
+        launched: @launched
       )
     end
   end
@@ -171,7 +170,7 @@ class Pgtk::Stash
   private
 
   def launch!
-    raise 'Cannot launch multiple times on same cache data' unless @stash[:launched].make_true
+    raise 'Cannot launch multiple times on same cache data' unless @launched.make_true
     Concurrent::FixedThreadPool.new(@threads).then do |threadpool|
       Concurrent::TimerTask.execute(execution_interval: 60 * 60, executor: threadpool) do
         @entrance.with_write_lock do
@@ -193,10 +192,12 @@ class Pgtk::Stash
           @stash[:queries][q].each_key do |k|
             next unless @stash[:queries][q][k][:stale]
             threadpool.post do
+              h = @stash[:queries][q][k]
+              ret = @pool.exec(q, h[:params], h[:result])
               @entrance.with_write_lock do
                 h = @stash[:queries][q][k]
                 h[:stale] = false
-                h[:ret] = @pool.exec(q, h[:params], h[:result])
+                h[:ret] = ret
               end
             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.21.3'
+  VERSION = '0.21.4'
 end

metadata

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