Removed unused options parameter

In the following cases we were passing options around but never using
them, introduced in the original implementation but never used. I found
them misleading as whatever you passed in would ultimately be ignored.

I've removed them so the code is more explicit and hopefully a little
less confusing.

Changelog.md

@@ -9,6 +9,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 ### Removed
 - Removed the `dry_run` option (https://github.com/zendesk/global_uid/pull/64)
 - Removed `GlobalUid::ServerVariables` module (https://github.com/zendesk/global_uid/pull/66)
+- Removed `options` parameter from `generate_uid` & `generate_many_uids` (https://github.com/zendesk/global_uid/pull/68)
 
 ## [3.7.1] - 2020-02-06
 ### Added

lib/global_uid/active_record_extension.rb

@@ -27,12 +27,12 @@ def global_uid_disabled
         @global_uid_disabled
       end
 
-      def generate_uid(options = {})
-        GlobalUid::Base.get_uid_for_class(self, options)
+      def generate_uid
+        GlobalUid::Base.get_uid_for_class(self)
       end
 
-      def generate_many_uids(count, options = {})
-        GlobalUid::Base.get_many_uids_for_class(self, count, options)
+      def generate_many_uids(count)
+        GlobalUid::Base.get_many_uids_for_class(self, count)
       end
 
       def disable_global_uid

lib/global_uid/base.rb

@@ -44,7 +44,7 @@ def self.create_uid_tables(id_table_name, options={})
       end
     end
 
-    def self.drop_uid_tables(id_table_name, options={})
+    def self.drop_uid_tables(id_table_name)
       with_connections do |connection|
         connection.execute("DROP TABLE IF EXISTS `#{id_table_name}`")
       end
@@ -70,7 +70,7 @@ def self.new_connection(name, connection_timeout)
       end
     end
 
-    def self.init_server_info(options)
+    def self.init_server_info
       id_servers = self.global_uid_servers
       increment_by = self.global_uid_options[:increment_by]
 
@@ -92,12 +92,11 @@ def self.disconnect!
       self.servers = nil
     end
 
-    def self.setup_connections!(options)
+    def self.setup_connections!
       connection_timeout = self.global_uid_options[:connection_timeout]
-      increment_by       = options[:increment_by]
 
       if self.servers.nil?
-        self.servers = init_server_info(options)
+        self.servers = init_server_info
         # sorting here sets up each process to have affinity to a particular server.
         self.servers = self.servers.sort_by { |s| s[:rand] }
       end
@@ -117,9 +116,8 @@ def self.setup_connections!(options)
       self.servers
     end
 
-    def self.with_connections(options = {})
-      options = self.global_uid_options.merge(options)
-      servers = setup_connections!(options)
+    def self.with_connections
+      servers = setup_connections!
 
       if !self.global_uid_options[:per_process_affinity]
         servers = servers.sort_by { rand } #yes, I know it's not true random.
@@ -156,11 +154,11 @@ def self.notify(exception, message)
       end
     end
 
-    def self.get_connections(options = {})
+    def self.get_connections
       with_connections {}
     end
 
-    def self.get_uid_for_class(klass, options = {})
+    def self.get_uid_for_class(klass)
       with_connections do |connection|
         Timeout.timeout(self.global_uid_options[:query_timeout], TimeoutException) do
           id = connection.insert("REPLACE INTO #{klass.global_uid_table} (stub) VALUES ('a')")
@@ -170,7 +168,7 @@ def self.get_uid_for_class(klass, options = {})
       raise NoServersAvailableException, "All global UID servers are gone!"
     end
 
-    def self.get_many_uids_for_class(klass, count, options = {})
+    def self.get_many_uids_for_class(klass, count)
       return [] unless count > 0
       with_connections do |connection|
         Timeout.timeout(self.global_uid_options[:query_timeout], TimeoutException) do

lib/global_uid/migration_extension.rb

@@ -25,7 +25,7 @@ def create_table(name, options = {}, &blk)
     def drop_table(name, options = {})
       if !GlobalUid::Base.global_uid_options[:disabled] && options[:use_global_uid] == true
         id_table_name = options[:global_uid_table] || GlobalUid::Base.id_table_from_name(name)
-        GlobalUid::Base.drop_uid_tables(id_table_name,options)
+        GlobalUid::Base.drop_uid_tables(id_table_name)
       end
       super(name, options)
     end