flipper-active_record 0.26.1 → 0.27.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5244fa15983cbfcba71bcf8d555d71d3e9142314d035dc7dad084ea413d0abfd
-  data.tar.gz: 810a08e5c66d66765f8a09e5b4554804792200b0fa8084fd38bb12a2db44cb08
+  metadata.gz: f1ec7d9f3d54551e8b6085d9105c854f96dfe8f6ac0d9b7840652e7f2882ee3c
+  data.tar.gz: 8d28946c836b4b405f522b4ae703659cdd8ba4194024098ffc9d4a9239821744
 SHA512:
-  metadata.gz: 20d45ea8dc75cbab5be320ea9ca22487f7a15483088c2370d48bb0e1b01e1fbf6cb657f9cd3f17ffd02c4fbeef1e20a80d3f328eb5a29d03961ddbfc87bd1439
-  data.tar.gz: 5d3871e886483ee2c9cefa90ef1fb918b4fd795c46121251b62c4c7543868578ba519f6f9547a58a749c87f4840e09496ef9c403f1765b91851be8bfd44c5d6b
+  metadata.gz: 772e0c2625811d8edfeed372c5a79ee3518328791e73ef86ad0dc4bc7093e9e53d2ae907b1a6a19be81a482e6f1f1ddd4edfe32a58b0f063e9c719d8c7f61083
+  data.tar.gz: 8023576edb27c613d508884adf287e04f8305c20b7d8e0c36718fa37979e122da6342234db984751c05fb607dd1d5151048913ff25e2751d4c08acff0850e083

data/benchmark/active_record_adapter_ips.rb

@@ -0,0 +1,15 @@
+require 'bundler/setup'
+require_relative './active_record_setup'
+require 'flipper'
+require 'flipper/adapters/active_record'
+require 'benchmark/ips'
+
+flipper = Flipper.new(Flipper::Adapters::ActiveRecord.new)
+
+2000.times do |i|
+  flipper.enable_actor :foo, Flipper::Actor.new("User;#{i}")
+end
+
+Benchmark.ips do |x|
+  x.report("get_all") { flipper.preload_all }
+end

data/benchmark/active_record_ips.rb

@@ -0,0 +1,17 @@
+require 'bundler/setup'
+require_relative './active_record_setup'
+require 'flipper'
+require 'flipper/adapters/active_record'
+require 'benchmark/ips'
+
+flipper = Flipper.new(Flipper::Adapters::ActiveRecord.new)
+
+2000.times do |i|
+  flipper.enable_actor :foo, Flipper::Actor.new("User;#{i}")
+end
+
+Benchmark.ips do |x|
+  x.report("all") { Flipper::Adapters::ActiveRecord::Gate.where(feature_key: "foo".freeze).load }
+  x.report("pluck") { Flipper::Adapters::ActiveRecord::Gate.where(feature_key: "foo".freeze).pluck(:key, :value) }
+  x.compare!
+end

data/benchmark/active_record_setup.rb

@@ -0,0 +1,31 @@
+require 'bundler/setup'
+require 'active_record'
+
+ActiveRecord::Base.establish_connection({
+  adapter: 'sqlite3',
+  database: ':memory:',
+})
+
+ActiveRecord::Base.connection.execute <<-SQL
+  CREATE TABLE flipper_features (
+    id integer PRIMARY KEY,
+    key text NOT NULL UNIQUE,
+    created_at datetime NOT NULL,
+    updated_at datetime NOT NULL
+  )
+SQL
+
+ActiveRecord::Base.connection.execute <<-SQL
+  CREATE TABLE flipper_gates (
+    id integer PRIMARY KEY,
+    feature_key text NOT NULL,
+    key text NOT NULL,
+    value text DEFAULT NULL,
+    created_at datetime NOT NULL,
+    updated_at datetime NOT NULL
+  )
+SQL
+
+ActiveRecord::Base.connection.execute <<-SQL
+  CREATE UNIQUE INDEX index_gates_on_keys_and_value on flipper_gates (feature_key, key, value)
+SQL

data/examples/active_record/cached.rb

@@ -0,0 +1,23 @@
+require_relative "./ar_setup"
+
+# Requires the flipper-active_record gem to be installed.
+require 'flipper/adapters/active_record'
+require 'flipper/adapters/active_support_cache_store'
+
+Flipper.configure do |config|
+  config.adapter do
+    Flipper::Adapters::ActiveSupportCacheStore.new(
+      Flipper::Adapters::ActiveRecord.new,
+      ActiveSupport::Cache::MemoryStore.new,
+      expires_in: 2.seconds
+    )
+  end
+end
+
+ActiveRecord::Base.logger = Logger.new(STDOUT)
+
+puts "You should see 5 or 6 queries."
+10.times do |i|
+  Flipper.enabled?(:foo)
+  sleep 1
+end

data/lib/flipper/adapters/active_record.rb

@@ -95,17 +95,22 @@ module Flipper
       #
       # Returns a Hash of Flipper::Gate#key => value.
       def get(feature)
-        db_gates = with_connection(@gate_class) { @gate_class.where(feature_key: feature.key) }
-        result_for_feature(feature, db_gates)
+        gates = with_connection(@gate_class) { @gate_class.where(feature_key: feature.key).pluck(:key, :value) }
+        result_for_gates(feature, gates)
       end
 
       def get_multi(features)
         with_connection(@gate_class) do
-          db_gates = @gate_class.where(feature_key: features.map(&:key))
-          grouped_db_gates = db_gates.group_by(&:feature_key)
+          gates = @gate_class.where(feature_key: features.map(&:key)).pluck(:feature_key, :key, :value)
+          grouped_gates = gates.inject({}) do |hash, (feature_key, key, value)|
+            hash[feature_key] ||= []
+            hash[feature_key] << [key, value]
+            hash
+          end
+
           result = {}
           features.each do |feature|
-            result[feature.key] = result_for_feature(feature, grouped_db_gates[feature.key])
+            result[feature.key] = result_for_gates(feature, grouped_gates[feature.key])
           end
           result
         end
@@ -113,18 +118,26 @@ module Flipper
 
       def get_all
         with_connection(@feature_class) do
+          # query the gates from the db in a single query
           features = ::Arel::Table.new(@feature_class.table_name.to_sym)
           gates = ::Arel::Table.new(@gate_class.table_name.to_sym)
-          rows_query = features.join(gates, Arel::Nodes::OuterJoin)
+          rows_query = features.join(gates, ::Arel::Nodes::OuterJoin)
             .on(features[:key].eq(gates[:feature_key]))
             .project(features[:key].as('feature_key'), gates[:key], gates[:value])
-          rows = @feature_class.connection.select_all rows_query
-          db_gates = rows.map { |row| @gate_class.new(row) }
-          grouped_db_gates = db_gates.group_by(&:feature_key)
+          gates = @feature_class.connection.select_rows(rows_query)
+
+          # group the gates by feature key
+          grouped_gates = gates.inject({}) do |hash, (feature_key, key, value)|
+            hash[feature_key] ||= []
+            hash[feature_key] << [key, value]
+            hash
+          end
+
+          # build up the result hash
           result = Hash.new { |hash, key| hash[key] = default_config }
-          features = grouped_db_gates.keys.map { |key| Flipper::Feature.new(key, self) }
+          features = grouped_gates.keys.map { |key| Flipper::Feature.new(key, self) }
           features.each do |feature|
-            result[feature.key] = result_for_feature(feature, grouped_db_gates[feature.key])
+            result[feature.key] = result_for_gates(feature, grouped_gates[feature.key])
           end
           result
         end
@@ -219,22 +232,22 @@ module Flipper
         # already added so no need move on with life
       end
 
-      def result_for_feature(feature, db_gates)
-        db_gates ||= []
+      def result_for_gates(feature, gates)
         result = {}
+        gates ||= []
         feature.gates.each do |gate|
           result[gate.key] =
             case gate.data_type
             when :boolean
-              if detected_db_gate = db_gates.detect { |db_gate| db_gate.key == gate.key.to_s }
-                detected_db_gate.value
+              if row = gates.detect { |key, value| !key.nil? && key.to_sym == gate.key }
+                row.last
               end
             when :integer
-              if detected_db_gate = db_gates.detect { |db_gate| db_gate.key == gate.key.to_s }
-                detected_db_gate.value
+              if row = gates.detect { |key, value| !key.nil? && key.to_sym == gate.key }
+                row.last
               end
             when :set
-              db_gates.select { |db_gate| db_gate.key == gate.key.to_s }.map(&:value).to_set
+              gates.select { |key, value| !key.nil? && key.to_sym == gate.key }.map(&:last).to_set
             else
               unsupported_data_type gate.data_type
             end

data/lib/flipper/version.rb

@@ -1,3 +1,3 @@
 module Flipper
-  VERSION = '0.26.1'.freeze
+  VERSION = '0.27.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flipper-active_record
 version: !ruby/object:Gem::Version
-  version: 0.26.1
+  version: 0.27.0
 platform: ruby
 authors:
 - John Nunemaker
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-15 00:00:00.000000000 Z
+date: 2023-03-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flipper
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.26.1
+        version: 0.27.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.26.1
+        version: 0.27.0
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
@@ -51,8 +51,12 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- benchmark/active_record_adapter_ips.rb
+- benchmark/active_record_ips.rb
+- benchmark/active_record_setup.rb
 - examples/active_record/ar_setup.rb
 - examples/active_record/basic.rb
+- examples/active_record/cached.rb
 - examples/active_record/internals.rb
 - flipper-active_record.gemspec
 - lib/flipper-active_record.rb