post_json 1.0.6 → 1.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 96d7ef19a5b074957629924c45a9a71894cc5f97
-  data.tar.gz: ca189492fea7bf032db65d2762c852b66c39532b
+  metadata.gz: 2f25ad4b6cb31d0b679bfa6c41d8b5533c3c69a8
+  data.tar.gz: 25b074ed6292d61272a2b3bb08c021d32ca881af
 SHA512:
-  metadata.gz: 263da488e04ee6c7dc08aaaed72d21c0802bc3d6ec96aa5575e0519c8945c72c31f20c346e432eaa182cfe8b1b964f5bcd0eeacdca7f02b3d693606ae73cc8d1
-  data.tar.gz: b3a221adbf2b1b67c810a0c0b7c76c942ff4f68dc3a3234e75557a7006c07ce49be538560218884085f94ff08e8108a8050464cee7e6b1b35cee78c7edf6d389
+  metadata.gz: 9783c9e9b72e4d5f6fadd58718a25c3a13c29a1b6168997f1abbbb7511c21d5891badde842b51b70885b19a077d63d5595acc891a7e486f464eb9bf574d9667f
+  data.tar.gz: 71267eaa0731c1a4340ec28808fcbcd3961ccddb36b5525170bc3b85727fac0d9d78d22f96ae2133c82758462c6d1ec6af385472b8de03c3c7f53e4eacd3d327

data/README.md

@@ -135,7 +135,32 @@ find_by!, find_each, find_in_batches, first, first!, first_or_create, first_or_i
 many?, offset, only, order, pluck, reorder, reverse_order, select, size, take, take!, to_a, to_sql, and where.
         
 We also added `page(page, per_page)`, which translate into `offset((page-1)*per_page).limit(per_page)`.
+
+
+## Performance
+
+On a virtual machine running on a 3 year old laptop we created 100.000 documents:
+
+        test_model = PostJson::Collection["test"]
+        100000.times { test_model.create(content: SecureRandom.uuid) }
+        content = test_model.last.content
         
+        result = test_model.where(content: content).count
+        # Rails debug tells me the duration was 975.5ms
+
+The duration was above 50ms as you can see.
+
+PostJson has a feature called "Dynamic Index". It is enabled by default and works automatic behind the scene. It has now created an index on 'content'.
+    
+Now lets see how the performance will be on the second and future queries using 'content':
+
+        result = test_model.where(content: content).count
+        # Rails debug tells me the duration was 1.5ms
+
+It shows PostgreSQL as a document database combined with indexing has great performance out of the box.
+
+See the next section about "Dynamic Indexes" for details.
+
 ## Dynamic Indexes
 
 Most applications do the same queries over and over again. This is why we think it is useful, if PostJson create indexes on slow queries.
@@ -171,7 +196,29 @@ You can adjust the settings:
 
 You might already know this about User Interfaces, but it is usual considered good practice if auto-complete responses are served to the user within 100 milliseconds. Other results are usual okay within 500 milliseconds. So leave room for application processing and network delay.
 
-Do not set create_dynamic_index_milliseconds_threshold too low as PostJson will try to create an index for every query performance. Like a threshold of 1 millisecond will be less than almost all query durations.
+Do not set create_dynamic_index_milliseconds_threshold too low as PostJson will try to create an index for every query performance. Like a threshold of 1 millisecond will be less than the duration of almost all queries.
+
+## Primary Keys
+
+PostJson assign UUID as primary key (id):
+
+        me = Person.create(name: "Jacob")
+        puts me.id
+        # "297a2500-a456-459b-b3e9-e876f59602c2"
+
+But you also set the primary key yourself:
+
+        john_doe = Person.create(id: "John Doe")
+
+Notice the primary key is downcased when doing a query or finding records:
+
+        found = Person.where(id: "JOhN DoE").first
+        puts found
+        # {"id"=>"John Doe", "version"=>1, "created_at"=>"2013-10-22T10:42:26.190Z", "updated_at"=>"2013-10-22T10:42:26.190Z"}
+        
+        found_again = Person.find("JOhN DoE")
+        puts found_again.attributes
+        # {"id"=>"John Doe", "version"=>1, "created_at"=>"2013-10-22T10:42:26.190Z", "updated_at"=>"2013-10-22T10:42:26.190Z"}
 
 ## The future
 

data/lib/post_json.rb

@@ -6,6 +6,7 @@ require 'post_json/concerns/query_methods'
 require 'post_json/concerns/finder_methods'
 require 'post_json/concerns/settings_methods'
 require 'post_json/concerns/dynamic_index_methods'
+require 'post_json/concerns/copyable'
 require 'post_json/query_translator'
 require 'post_json/base'
 require 'post_json/model_settings'

data/lib/post_json/base.rb

@@ -9,6 +9,7 @@ module PostJson
 
     include SettingsMethods
     include DynamicIndexMethods
+    include Copyable
 
     def initialize(*args)
       __local__primary_key = self.class.primary_key

data/lib/post_json/concerns/copyable.rb

@@ -0,0 +1,36 @@
+module PostJson
+  module Copyable
+    extend ActiveSupport::Concern
+
+    module ClassMethods
+      def copy(destination_collection_name)
+        destination = Collection[destination_collection_name]
+        if exists?
+          src_id = persisted_settings.id
+
+          if destination.persisted?
+            dest_id = destination.persisted_settings.id
+            query = original_all
+            query = query.joins("INNER JOIN #{table_name} as dest ON dest.id = #{table_name}.id")
+            query = query.where("dest.__doc__model_settings_id = '#{dest_id}'")
+            query = query.where("\"#{table_name}\".__doc__model_settings_id = '#{src_id}'")
+            query = query.where("dest.id = \"#{table_name}\".id")
+            conflicting_ids = query.pluck("dest.id").join(", ")
+            if conflicting_ids.present?
+              error_message = "Following primary keys (#{primary_key}) already exists in collection \"#{destination.collection_name}\": #{conflicting_ids}."
+              raise ActiveRecord::RecordNotUnique, error_message
+            end
+          end
+
+          dest_id = destination.persisted_settings.id
+          selectors = PostJson::Base.column_names.map { |s| s == "__doc__model_settings_id" ? "'#{dest_id}' as #{s}" : s }.join(", ")
+          condition = "__doc__model_settings_id = '#{src_id}'"
+          destination.transaction do
+            destination.connection.execute("INSERT INTO #{table_name} (SELECT #{selectors} FROM #{table_name} WHERE #{condition})")
+          end
+        end
+        destination
+      end
+    end
+  end
+end

data/lib/post_json/concerns/finder_methods.rb

@@ -37,7 +37,8 @@ module PostJson
     end
 
     def destroy(id)
-      execute { |documents| documents.destroy(id) }
+      value = id ? id.to_s.downcase : nil
+      execute { |documents| documents.destroy(value) }
     end
 
     def destroy_all(conditions = nil)
@@ -62,7 +63,18 @@ module PostJson
     end
 
     def find(*args)
-      execute { |documents| documents.find(*args) }
+      if args.length == 0
+        execute { |documents| documents.find }
+      elsif args.length == 1 && args[0].is_a?(Array)
+        ids = args[0].map{|id| id ? id.to_s.downcase : nil}
+        execute { |documents| documents.find(ids) }
+      elsif args.length == 1 && args[0].is_a?(Array) == false
+        id = args[0] ? args[0].to_s.downcase : nil
+        execute { |documents| documents.find(id) }
+      else
+        ids = args.map{|id| id ? id.to_s.downcase : nil}
+        execute { |documents| documents.find(ids) }
+      end
     end
 
     def find_by(*args)
@@ -214,8 +226,8 @@ module PostJson
 
     def define_selector(attribute_name)
       case attribute_name.to_s
-      when "id"
-        "\"#{table_name}\".id"
+      when primary_key
+        "\"#{table_name}\".#{primary_key}"
       else
         "json_selector('#{attribute_name}', \"#{table_name}\".__doc__body)"
       end
@@ -248,17 +260,22 @@ module PostJson
             [:where, arguments]
           when :where_equal
             selector = define_selector(arguments[:attribute])
+            selector_is_primary_key = arguments[:attribute].to_s == primary_key
             argument = arguments[:argument]
             case argument
             when Array
               values = argument.map{|v| v ? v.to_s : nil}
+              values = values.map{|v| v.try(:downcase)} if selector_is_primary_key
               [:where, "(#{selector} IN (?))", values]
             when Range
               first_value = argument.first ? argument.first.to_s : nil
+              first_value = first_value.try(:downcase) if selector_is_primary_key
               last_value = argument.last ? argument.last.to_s : nil
+              last_value = last_value.try(:downcase) if selector_is_primary_key
               [:where, "(#{selector} BETWEEN ? AND ?)", first_value, last_value]
             else
               value = argument ? argument.to_s : nil
+              value = value.try(:downcase) if selector_is_primary_key
               [:where, "#{selector} = ?", value]
             end
           else

data/lib/post_json/concerns/settings_methods.rb

@@ -21,6 +21,14 @@ module PostJson
         settings
       end
 
+      def new?
+        settings.new_record?
+      end
+
+      def persisted?
+        settings.persisted?
+      end
+
       def reload_settings!
         @settings = nil
         settings

data/lib/post_json/query_translator.rb

@@ -11,6 +11,10 @@ module PostJson
       @relation.klass
     end
 
+    def primary_key
+      model_class.primary_key
+    end
+
     def table_name
       model_class.table_name
     end

data/lib/post_json/version.rb

@@ -1,3 +1,3 @@
 module PostJson
-  VERSION = "1.0.6"
+  VERSION = "1.0.7"
 end

data/spec/models/base_spec.rb

@@ -48,8 +48,13 @@ describe "Base model" do
 
   context "should downcase primary key" do
     let(:new_id) { "abc" }
-    subject { PostJson::Collection['Customer'].create id: "#{new_id.upcase}" }
+    let(:model) { PostJson::Collection['Customer'] }
+    let!(:record) { model.create id: "#{new_id.upcase}" }
+    subject { record }
     its(:id) { should == new_id }
+    it { model.exists?(id: "#{new_id.upcase}").should be_true }
+    it { model.where(id: "#{new_id.upcase}").count.should == 1 }
+    it { model.find("#{new_id.upcase}").id.should == new_id }
   end
 
   context "should now allow change of primary key" do
@@ -390,4 +395,42 @@ describe "Base model" do
     it { subject["updated_at"].should == document.updated_at.strftime('%Y-%m-%dT%H:%M:%S.%LZ') }
     it { subject["version"].should == document.version }
   end
+
+  context "copy" do
+    let(:orig_model) { PostJson::Collection["Originals"] }
+    let(:copy_model) { orig_model.copy("Copies") }
+    before do
+      orig_model.create(id: "Jacob", age: 33)
+      orig_model.create(id: "Martin", age: 29)
+    end
+
+    subject { copy_model }
+
+    its(:count) { should == 2 }
+    it { subject.find("Jacob").age.should == 33 }
+    it { subject.find("Martin").age.should == 29 }
+
+    context "conflict" do
+      before do
+        copy_model
+        orig_model.create(id: "Jonathan", age: 33)
+      end
+      it { expect { orig_model.copy(copy_model.collection_name) }.to raise_error(ActiveRecord::RecordNotUnique) }
+    end
+
+    context "to existing collection" do
+      let(:other_model) { PostJson::Collection["Others"] }
+      before do
+        other_model.create(id: "John", age: 25)
+        other_model.copy(copy_model.collection_name)
+        orig_model.create(id: "Jonathan", age: 33)
+      end
+      subject { copy_model }
+
+      its(:count) { should == 3 }
+      it { subject.find("Jacob").age.should == 33 }
+      it { subject.find("Martin").age.should == 29 }
+      it { subject.find("John").age.should == 25 }
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: post_json
 version: !ruby/object:Gem::Version
-  version: 1.0.6
+  version: 1.0.7
 platform: ruby
 authors:
 - Jacob Madsen and Martin Thoegersen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-20 00:00:00.000000000 Z
+date: 2013-10-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -103,6 +103,7 @@ files:
 - lib/post_json/concerns/settings_methods.rb
 - lib/post_json/concerns/query_methods.rb
 - lib/post_json/concerns/argument_methods.rb
+- lib/post_json/concerns/copyable.rb
 - lib/post_json/concerns/finder_methods.rb
 - lib/post_json/concerns/dynamic_index_methods.rb
 - lib/post_json/version.rb