switchman 1.8.1 → 1.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 50222908649b926ff4aadbe563d555b8e08bb6ef
-  data.tar.gz: 1ed9670ef0a8e48243f9089f27c8defe49f95f51
+  metadata.gz: d2e63c8b9d8a43c1cf8716f775593bc0e5c7296b
+  data.tar.gz: c4cda98c0bbdf18d0c648adcf8d8465c90f2246e
 SHA512:
-  metadata.gz: 96128751d37b322f381227b2c4326f683a34f1700853001f3a0071c033f584e204677c1d836c129c86cb856005bb321263e9ad5af67160cc7a551a909d894e59
-  data.tar.gz: 3cda176ecdd6c7857b6e8d1bdfef3c43b08123d665fce3c3d35c2a11c007cc706ee47079e2755192d9a21d6112808bc403fe9e87e259c52c24aa4e5b0aefc68d
+  metadata.gz: f2e7e0889dce714d0d128eeca5b09434ba0cd313ebe9e09db2f1d58e7d706e6aa1353e03b54496d2f145fbd19467c74bd095870025eb6fca0321170c06ffd14a
+  data.tar.gz: e0614890e323dcedd8b272e1ec571be80de1640d27601d9c905208e4fca69931676ec03e4ef038775aa634e43751909bdf0b18f261fe32b1c8a394764c830d24

data/db/migrate/20161206323434_add_back_default_string_limits_switchman.rb

@@ -0,0 +1,11 @@
+class AddBackDefaultStringLimitsSwitchman < ActiveRecord::Migration
+  def up
+    add_string_limit_if_missing :switchman_shards, :name
+    add_string_limit_if_missing :switchman_shards, :database_server_id
+  end
+
+  def add_string_limit_if_missing(table, column)
+    return if column_exists?(table, column, :string, limit: 255)
+    change_column table, column, :string, limit: 255
+  end
+end

data/lib/switchman/active_record/calculations.rb

@@ -1,16 +1,13 @@
 module Switchman
   module ActiveRecord
     module Calculations
-      CALL_SUPER = Object.new.freeze
-      private_constant :CALL_SUPER
 
       def pluck(*column_names)
-        return super(*column_names[1..-1]) if column_names.first.equal?(CALL_SUPER)
         target_shard = Shard.current(klass.shard_category)
         shard_count = 0
         result = self.activate do |relation, shard|
           shard_count += 1
-          results = relation.pluck(CALL_SUPER, *column_names)
+          results = relation.call_super(:pluck, Calculations, *column_names)
           if column_names.length > 1
             column_names.each_with_index do |column_name, idx|
               if klass.sharded_column?(column_name)
@@ -30,13 +27,12 @@ module Switchman
         result
       end
 
-      def execute_simple_calculation(operation, column_name, distinct, super_method: false)
-        return super(operation, column_name, distinct) if super_method
+      def execute_simple_calculation(operation, column_name, distinct)
         operation = operation.to_s.downcase
         if operation == "average"
           result = calculate_simple_average(column_name, distinct)
         else
-          result = self.activate{ |relation| relation.send(:execute_simple_calculation, operation, column_name, distinct, super_method: true) }
+          result = self.activate{ |relation| relation.call_super(:execute_simple_calculation, Calculations, operation, column_name, distinct) }
           if result.is_a?(Array)
             case operation
             when "count", "sum"

data/lib/switchman/active_record/finder_methods.rb

@@ -1,9 +1,8 @@
 module Switchman
   module ActiveRecord
     module FinderMethods
-      def find_one(id, call_super: false)
+      def find_one(id)
         return super(id) unless klass.integral_id?
-        return super(id) if call_super
 
         if shard_source_value != :implicit
           current_shard = Shard.current(klass.shard_category)
@@ -14,7 +13,7 @@ module Switchman
             # skip the shard if the object can't be on it. unless we're only looking at one shard;
             # we might be expecting a shadow object
             next if current_id > Shard::IDS_PER_SHARD && self.all_shards.length > 1
-            relation.send(:find_one, current_id, call_super: true)
+            relation.call_super(:find_one, FinderMethods, current_id)
           end
           if result.is_a?(Array)
             result = result.first
@@ -52,9 +51,9 @@ module Switchman
           relation = relation.where(table[primary_key].eq(conditions)) if conditions != :none
         end
 
-        args = [relation, "#{name} Exists"]
-        args << relation.bind_values
-        relation.activate { return true if connection.select_value(*args) }
+        relation.activate do |shard_rel|
+          return true if connection.select_value(shard_rel, "#{name} Exists", shard_rel.bind_values)
+        end
         false
       end
     end

data/lib/switchman/active_record/relation.rb

@@ -42,17 +42,15 @@ module Switchman
         primary_shard.activate(klass.shard_category) { super }
       end
 
-      def explain(super_method: false)
-        return super() if super_method
-        self.activate { |relation| relation.explain(super_method: true) }
+      def explain
+        self.activate { |relation| relation.call_super(:explain, Relation) }
       end
 
       to_a_method = ::Rails.version >= '5' ? :records : :to_a
       class_eval <<-RUBY, __FILE__, __LINE__ + 1
-        def #{to_a_method}(super_method: false)
-          return super() if super_method
+        def #{to_a_method}
           return @records if loaded?
-          results = self.activate { |relation| relation.#{to_a_method}(super_method: true) }
+          results = self.activate { |relation| relation.call_super(#{to_a_method.inspect}, Relation) }
           case shard_value
           when Array, ::ActiveRecord::Relation, ::ActiveRecord::Base
             @records = results
@@ -62,14 +60,10 @@ module Switchman
         end
       RUBY
 
-      CALL_SUPER = Object.new.freeze
-      private_constant :CALL_SUPER
-
-      %w{update_all delete_all}.each do |method|
+      %I{update_all delete_all}.each do |method|
         class_eval <<-RUBY, __FILE__, __LINE__ + 1
           def #{method}(*args)
-            return super(*args[1..-1]) if args.first.equal?(CALL_SUPER)
-            result = self.activate { |relation| relation.#{method}(CALL_SUPER, *args) }
+            result = self.activate { |relation| relation.call_super(#{method.inspect}, Relation, *args) }
             result = result.sum if result.is_a?(Array)
             result
           end

data/lib/switchman/call_super.rb

@@ -0,0 +1,18 @@
+module Switchman
+  module CallSuper
+    def super_method_above(method_name, above_module)
+      @super_methods ||= {}
+      @super_methods[[method_name, above_module]] ||= begin
+        method = method(method_name)
+        while method.owner != above_module
+          method = method.super_method
+        end
+        method.super_method
+      end
+    end
+
+    def call_super(method, above_module, *args, &block)
+      super_method_above(method, above_module).call(*args, &block)
+    end
+  end
+end

data/lib/switchman/engine.rb

@@ -84,6 +84,7 @@ module Switchman
         require "switchman/active_record/type_caster"
         require "switchman/active_record/where_clause_factory"
         require "switchman/arel"
+        require "switchman/call_super"
         require "switchman/rails"
         require "switchman/shackles/relation"
         require "switchman/shard_internal"
@@ -134,9 +135,10 @@ module Switchman
         ::ActiveRecord::Relation.prepend(ActiveRecord::Calculations)
         ::ActiveRecord::Relation.include(ActiveRecord::FinderMethods)
         ::ActiveRecord::Relation.include(ActiveRecord::QueryMethods)
+        ::ActiveRecord::Relation.prepend(Shackles::Relation)
         ::ActiveRecord::Relation.prepend(ActiveRecord::Relation)
         ::ActiveRecord::Relation.include(ActiveRecord::SpawnMethods)
-        ::ActiveRecord::Relation.prepend(Shackles::Relation)
+        ::ActiveRecord::Relation.include(CallSuper)
 
         if ::Rails.version >= '5'
           ::ActiveRecord::Relation::WhereClauseFactory.prepend(ActiveRecord::WhereClauseFactory)

data/lib/switchman/version.rb

@@ -1,3 +1,3 @@
 module Switchman
-  VERSION = "1.8.1"
+  VERSION = "1.9.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: switchman
 version: !ruby/object:Gem::Version
-  version: 1.8.1
+  version: 1.9.0
 platform: ruby
 authors:
 - Cody Cutrer
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-11-23 00:00:00.000000000 Z
+date: 2016-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -190,6 +190,7 @@ files:
 - app/models/switchman/shard_internal.rb
 - db/migrate/20130328212039_create_switchman_shards.rb
 - db/migrate/20130328224244_create_default_shard.rb
+- db/migrate/20161206323434_add_back_default_string_limits_switchman.rb
 - lib/switchman.rb
 - lib/switchman/action_controller/caching.rb
 - lib/switchman/active_record/abstract_adapter.rb
@@ -217,6 +218,7 @@ files:
 - lib/switchman/active_record/where_clause_factory.rb
 - lib/switchman/active_support/cache.rb
 - lib/switchman/arel.rb
+- lib/switchman/call_super.rb
 - lib/switchman/connection_pool_proxy.rb
 - lib/switchman/database_server.rb
 - lib/switchman/default_shard.rb
@@ -246,7 +248,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.0'
+      version: '2.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="