frozen_record 0.26.0 → 0.26.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 558032391d508fe0e51074f70abb81395189376a610c694cf6b43a64bbf2b91a
-  data.tar.gz: e8bd4b3bb7e2dafb6d3854176b7c98e6f87a3148790dd5d5f0de0ec55b95ef48
+  metadata.gz: 4e4a597e4419873da69776c887c5ef35c812f785493cd9597d3d3ced891578ab
+  data.tar.gz: d46b86085ceeee261b39a281aab1c62a2551e872ad1f4696e8a7325b7cfa9a7a
 SHA512:
-  metadata.gz: 725c9abc9be65de3c1db5643e0df340c34b9b72f50e4a650d0f0a51a27160f6f7db6bcdbb33cab77d7fac3d9e7c6558fe22fd04081b43b8da4c6f7ba2887a3e4
-  data.tar.gz: a1f7e30104161efb60b5a6fbb89c1270f6656557c4f8bfb549a18ec012fdc87be73223e2d13d1efc73d18c3c8e2a0c905f9b9d0bb30907dcd465b659801f41fe
+  metadata.gz: 8e68a5dfb1d888de4f6dc72bb42ffdc1c08af87229b7bd0dcf107d1da9b48ef5b948adea5652b132cb497a1a8c15b513aa7ba8beea7c863b06f19de14ae02865
+  data.tar.gz: f787b5072efd3997fb41942c1d9b6014ee872d6faec12a4c5a77ef489a7039ed980987bbf4102ed7d9c5cd2b296e49c1bd6a25f578559969d200d875d6cdf877

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Unreleased
 
+# v0.26.1
+
+- Optimized single attribute lookup.
+
 # v0.26.0
 
 - Drop dependency on `dedup` gem.

data/benchmark/querying

@@ -23,6 +23,13 @@ class LargeCountry < Country
   end
 end
 
+LargeCountry.eager_load!
+
+puts "=== unique index lookup ==="
+Benchmark.ips do |x|
+  x.report('pk lookup') { LargeCountry.find_by(name: "Canada1") }
+end
+
 puts "=== simple scalar match ==="
 Benchmark.ips do |x|
   x.report('simple') { LargeCountry.nato.size }

data/lib/frozen_record/base.rb

@@ -107,7 +107,7 @@ module FrozenRecord
         store[:scope] = scope
       end
 
-      delegate :each, :find, :find_each, :find_by_id, :find_by, :find_by!, :where, :first, :first!, :last, :last!,
+      delegate :each, :find_each, :where, :first, :first!, :last, :last!,
                :pluck, :ids, :order, :limit, :offset, :minimum, :maximum, :average, :sum, :count,
                to: :current_scope
 
@@ -123,6 +123,31 @@ module FrozenRecord
         end
       end
 
+      def find_by_id(id)
+        find_by(primary_key => id)
+      end
+
+      def find(id)
+        raise RecordNotFound, "Can't lookup record without ID" unless id
+        find_by(primary_key => id) or raise RecordNotFound, "Couldn't find a record with ID = #{id.inspect}"
+      end
+
+      def find_by(criterias)
+        if criterias.size == 1
+          criterias.each do |attribute, value|
+            attribute = attribute.to_s
+            if index = index_definitions[attribute]
+              return index.lookup(value).first
+            end
+          end
+        end
+        current_scope.find_by(criterias)
+      end
+
+      def find_by!(criterias)
+        find_by(criterias) or raise RecordNotFound, "No record matched"
+      end
+
       def add_index(attribute, unique: false)
         index = unique ? UniqueIndex.new(self, attribute) : Index.new(self, attribute)
         self.index_definitions = index_definitions.merge(index.attribute => index).freeze

data/lib/frozen_record/scope.rb

@@ -34,12 +34,17 @@ module FrozenRecord
     end
 
     def find_by_id(id)
-      matching_records.find { |r| r.id == id }
+      find_by(@klass.primary_key => id)
     end
 
     def find(id)
       raise RecordNotFound, "Can't lookup record without ID" unless id
-      find_by_id(id) or raise RecordNotFound, "Couldn't find a record with ID = #{id.inspect}"
+
+      scope = self
+      if @limit || @offset
+        scope = limit(nil).offset(nil)
+      end
+      scope.find_by_id(id) or raise RecordNotFound, "Couldn't find a record with ID = #{id.inspect}"
     end
 
     def find_by(criterias)

data/lib/frozen_record/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FrozenRecord
-  VERSION = '0.26.0'
+  VERSION = '0.26.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: frozen_record
 version: !ruby/object:Gem::Version
-  version: 0.26.0
+  version: 0.26.1
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-07-12 00:00:00.000000000 Z
+date: 2022-08-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel