move the dispatch table to be per-instance

visitors are not shared among threads, so any mutations to the cache
should be OK.  The cache is also pre-populated on construction, but we
should pull that out so we can share the cache among visitors in the
future.

1 files changed, 4 insertions, 4 deletions

diff --git a/lib/arel/visitors/reduce.rb b/lib/arel/visitors/reduce.rb
index 9670cad27c..1d74934fe5 100644
--- a/lib/arel/visitors/reduce.rb
+++ b/lib/arel/visitors/reduce.rb
@@ -10,14 +10,14 @@ module Arel
       private
 
       def visit object, collector
-        send dispatch[object.class], object, collector
+        send dispatch[object.class.name], object, collector
       rescue NoMethodError => e
-        raise e if respond_to?(dispatch[object.class], true)
+        raise e if respond_to?(dispatch[object.class.name], true)
         superklass = object.class.ancestors.find { |klass|
-          respond_to?(dispatch[klass], true)
+          respond_to?(dispatch[klass.name], true)
         }
         raise(TypeError, "Cannot visit #{object.class}") unless superklass
-        dispatch[object.class] = dispatch[superklass]
+        dispatch[object.class.name] = dispatch[superklass.name]
         retry
       end
     end