switchman 3.6.7 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2e86d08703a350aa5b8d3d843c51a0813f00e6ee0c681d0c95094f84863b327b
-  data.tar.gz: f13e438d68d996f55c9be543cc8cb89967b62f9ddd111ab4fae188da8b959096
+  metadata.gz: eb5bc75d858d97e55442dfdafcd1826d7040fe1bd4819bc3c2ee6053257a72ab
+  data.tar.gz: e73beaf3a5987f26e2402078f66c028831afc6924f21546d0ca1cb35f70e0831
 SHA512:
-  metadata.gz: 7be63615d79be0b46184b9e6754bcb6091873e2eb0c4722705daea7dd81dd50320819b2238538c818637901570c2ad47a7ed62febd441c4d5b130c78391554cc
-  data.tar.gz: c3eb6290f9437ccac7162216840c2f258f31fc6c978121b9054c6308640bff6d522f2f61fd2247fea178c1c173a012a4d7f2c22b2e234d7b4f431ce4b8306403
+  metadata.gz: 273718d8bc773ded149ac5d81509a6d85c6858f17bc7c1a03f3dbdf15bbb96f101fea5842eddf87ba442aabf1f0bffe1a21421da4b9329548b0f40e51de26645
+  data.tar.gz: 9f81d1963fe213d334aed7377057fe29ecbdb637c6b2b8a33a49d52fcc0f726e1a8e87853a6e1030c8c2b0bbaff51e369d54cf8118f1c512ca0d62bf5684267c

data/lib/switchman/active_record/associations.rb

@@ -203,41 +203,7 @@ module Switchman
             # #compare_by_identity makes such owners different hash keys
             @records_by_owner = {}.compare_by_identity
 
-            if ::Rails.version >= "7.0"
-              raw_records ||= loader_query.records_for([self])
-            elsif owner_keys.empty?
-              raw_records ||= []
-            else
-              # determine the shard to search for each owner
-              if reflection.macro == :belongs_to
-                # for belongs_to, it's the shard of the foreign_key
-                partition_proc = lambda do |owner|
-                  if owner.class.sharded_column?(owner_key_name)
-                    Shard.shard_for(owner[owner_key_name], owner.shard)
-                  else
-                    Shard.current
-                  end
-                end
-              elsif !reflection.options[:multishard]
-                # for non-multishard associations, it's *just* the owner's shard
-                partition_proc = ->(owner) { owner.shard }
-              end
-
-              raw_records ||= Shard.partition_by_shard(owners, partition_proc) do |partitioned_owners|
-                relative_owner_keys = partitioned_owners.map do |owner|
-                  key = owner[owner_key_name]
-                  if key && owner.class.sharded_column?(owner_key_name)
-                    key = Shard.relative_id_for(key,
-                                                owner.shard,
-                                                Shard.current(klass.connection_class_for_self))
-                  end
-                  convert_key(key)
-                end
-                relative_owner_keys.compact!
-                relative_owner_keys.uniq!
-                records_for(relative_owner_keys)
-              end
-            end
+            raw_records ||= loader_query.records_for([self])
 
             @preloaded_records = raw_records.select do |record|
               assignments = false

data/lib/switchman/active_record/attribute_methods.rb

@@ -55,12 +55,7 @@ module Switchman
         end
 
         def define_cached_method(owner, name, namespace:, as:, &block)
-          if ::Rails.version < "7.0"
-            yield owner
-            owner.rename_method(as, name)
-          else
-            owner.define_cached_method(name, namespace: namespace, as: as, &block)
-          end
+          owner.define_cached_method(name, namespace: namespace, as: as, &block)
         end
 
         def define_method_global_attribute(attr_name, owner:)

data/lib/switchman/active_record/base.rb

@@ -136,11 +136,7 @@ module Switchman
         end
 
         def connected_to_stack
-          has_own_stack = if ::Rails.version < "7.0"
-                            Thread.current.thread_variable?(:ar_connected_to_stack)
-                          else
-                            ::ActiveSupport::IsolatedExecutionState.key?(:active_record_connected_to_stack)
-                          end
+          has_own_stack = ::ActiveSupport::IsolatedExecutionState.key?(:active_record_connected_to_stack)
 
           ret = super
           return ret if has_own_stack
@@ -189,12 +185,6 @@ module Switchman
 
           Shard.default
         end
-
-        if ::Rails.version < "7.0"
-          def connection_class_for_self
-            connection_classes
-          end
-        end
       end
 
       def self.prepended(klass)

data/lib/switchman/active_record/calculations.rb

@@ -148,19 +148,10 @@ module Switchman
       private
 
       def type_cast_calculated_value_switchman(value, column_name, operation)
-        if ::Rails.version < "7.0"
-          type_cast_calculated_value(value, operation) do |val|
-            column = aggregate_column(column_name)
-            type ||= column.try(:type_caster) ||
-                     lookup_cast_type_from_join_dependencies(column_name.to_s) || ::ActiveRecord::Type.default_value
-            type.deserialize(val)
-          end
-        else
-          column = aggregate_column(column_name)
-          type ||= column.try(:type_caster) ||
-                   lookup_cast_type_from_join_dependencies(column_name.to_s) || ::ActiveRecord::Type.default_value
-          type_cast_calculated_value(value, operation, type)
-        end
+        column = aggregate_column(column_name)
+        type ||= column.try(:type_caster) ||
+                 lookup_cast_type_from_join_dependencies(column_name.to_s) || ::ActiveRecord::Type.default_value
+        type_cast_calculated_value(value, operation, type)
       end
 
       def column_name_for(field)

data/lib/switchman/active_record/connection_pool.rb

@@ -60,7 +60,7 @@ module Switchman
       private
 
       def current_shard
-        (::Rails.version < "7.0") ? connection_klass.current_switchman_shard : connection_class.current_switchman_shard
+        connection_class.current_switchman_shard
       end
 
       def tls_key

data/lib/switchman/active_record/relation.rb

@@ -55,7 +55,7 @@ module Switchman
       end
 
       def load(&block)
-        if !loaded? || (::Rails.version >= "7.0" && scheduled?)
+        if !loaded? || scheduled?
           @records = activate { |relation| relation.send(:exec_queries, &block) }
           @loaded = true
         end

data/lib/switchman/engine.rb

@@ -10,20 +10,9 @@ module Switchman
 
     ::GuardRail.singleton_class.prepend(GuardRail::ClassMethods)
 
-    # after :initialize_dependency_mechanism to ensure autoloading is
-    # configured for any downstream initializers that care. In rails 7.0 we
-    # should be able to just use an explicit after on configuring the once
-    # autoloaders and not need to go monkey around with initializer order
-    if ::Rails.version < "7.0"
-      initialize_dependency_mechanism = ::Rails::Application::Bootstrap.initializers.find do |i|
-        i.name == :initialize_dependency_mechanism
-      end
-      initialize_dependency_mechanism.instance_variable_get(:@options)[:after] = :set_autoload_paths
-    end
-
     initializer "switchman.active_record_patch",
                 before: "active_record.initialize_database",
-                after: ((::Rails.version < "7.0") ? :initialize_dependency_mechanism : :setup_once_autoloader) do
+                after: :setup_once_autoloader do
       ::ActiveSupport.on_load(:active_record) do
         # Switchman requires postgres, so just always load the pg adapter
         require "active_record/connection_adapters/postgresql_adapter"
@@ -54,11 +43,9 @@ module Switchman
         ::ActiveRecord::Associations::CollectionProxy.include(ActiveRecord::Associations::CollectionProxy)
 
         ::ActiveRecord::Associations::Preloader::Association.prepend(ActiveRecord::Associations::Preloader::Association)
-        unless ::Rails.version < "7.0"
-          ::ActiveRecord::Associations::Preloader::Association::LoaderRecords.prepend(
-            ActiveRecord::Associations::Preloader::Association::LoaderRecords
-          )
-        end
+        ::ActiveRecord::Associations::Preloader::Association::LoaderRecords.prepend(
+          ActiveRecord::Associations::Preloader::Association::LoaderRecords
+        )
         ::ActiveRecord::ConnectionAdapters::AbstractAdapter.prepend(ActiveRecord::AbstractAdapter)
         unless ::Rails.version < "7.1"
           ::ActiveRecord::ConnectionAdapters::ConnectionHandler.prepend(ActiveRecord::ConnectionHandler)

data/lib/switchman/r_spec_helper.rb

@@ -126,6 +126,7 @@ module Switchman
             ds.shards.delete_all unless use_transactional_tests
             ds.destroy
           end
+          ds.remove_instance_variable(:@primary_shard_id) if ds.instance_variable_defined?(:@primary_shard_id)
         end
       end
 

data/lib/switchman/sharded_instrumenter.rb

@@ -16,11 +16,7 @@ module Switchman
         payload[:shard] = {
           database_server_id: shard.database_server.id,
           id: shard.id,
-          env: if ::Rails.version < "7.0"
-                 @shard_host.pool.connection_klass&.current_role
-               else
-                 @shard_host.pool.connection_class&.current_role
-               end
+          env: @shard_host.pool.connection_class&.current_role
         }
       end
       super(name, payload)

data/lib/switchman/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Switchman
-  VERSION = "3.6.7"
+  VERSION = "4.0.0"
 end

data/lib/tasks/switchman.rake

@@ -1,16 +1,5 @@
 # frozen_string_literal: true
 
-# In rails 7.0+ if you have only 1 db in the env it doesn't try to do explicit activation
-# (and for rails purposes we only have one db per env because each database server is a separate env)
-if Rails.version < "7.0"
-  task_prefix = Rake::Task.task_defined?("app:db:migrate") ? "app:db" : "db"
-  Rake::Task["#{task_prefix}:migrate"].clear_actions.enhance do
-    ActiveRecord::Tasks::DatabaseTasks.migrate
-    # Ensure this doesn't blow up when running inside the dummy app
-    Rake::Task["#{task_prefix}:_dump"].invoke
-  end
-end
-
 module Switchman
   module Rake
     def self.filter_database_servers
@@ -265,28 +254,9 @@ module Switchman
 
   module ActiveRecord
     module PostgreSQLDatabaseTasks
-      if ::Rails.version < "7.0"
-        def structure_dump(filename, extra_flags = nil)
-          set_psql_env
-          args = ["--schema-only", "--no-privileges", "--no-owner", "--file", filename]
-          args.concat(Array(extra_flags)) if extra_flags
-          shard = Shard.current.name
-          serialized_search_path = shard
-          args << "--schema=#{Shellwords.escape(shard)}"
-
-          ignore_tables = ::ActiveRecord::SchemaDumper.ignore_tables
-          args += ignore_tables.flat_map { |table| ["-T", table] } if ignore_tables.any?
-
-          args << db_config.database
-          run_cmd("pg_dump", args, "dumping")
-          remove_sql_header_comments(filename)
-          File.open(filename, "a") { |f| f << "SET search_path TO #{serialized_search_path};\n\n" }
-        end
-      else
-        def structure_dump(...)
-          ::ActiveRecord.dump_schemas = Switchman::Shard.current.name
-          super
-        end
+      def structure_dump(...)
+        ::ActiveRecord.dump_schemas = Switchman::Shard.current.name
+        super
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: switchman
 version: !ruby/object:Gem::Version
-  version: 3.6.7
+  version: 4.0.0
 platform: ruby
 authors:
 - Cody Cutrer
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-09-11 00:00:00.000000000 Z
+date: 2025-02-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -18,7 +18,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 6.1.4
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '7.2'
@@ -28,7 +28,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 6.1.4
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '7.2'
@@ -66,7 +66,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '7.2'
@@ -76,7 +76,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '7.2'