Rails.cache error in Rails 3.1 - TypeError: can't dump hash with default proc

后端 未结 4 964
滥情空心
滥情空心 2020-11-29 04:23

I running into an issue with the Rails.cache methods on 3.1.0.rc4 (ruby 1.9.2p180 (2011-02-18 revision 30909) [x86_64-darwin10]). The code works fine within the same applic

相关标签:
4条回答
  • 2020-11-29 04:49

    just remove the default proc after you finished altering it. something like:

    your_hash.default = nil # clear the default_proc
    
    0 讨论(0)
  • 2020-11-29 05:04

    This might be a little verbose but I had to spend some time with the Rails source code to learn how the caching internals work. Writing things down aids my understanding and I figure that sharing some notes on how things work can't hurt. Skip to the end if you're in a hurry.


    Why It Happens

    This is the offending method inside ActiveSupport:

    def should_compress?(value, options)
      if options[:compress] && value
        unless value.is_a?(Numeric)
          compress_threshold = options[:compress_threshold] || DEFAULT_COMPRESS_LIMIT
          serialized_value = value.is_a?(String) ? value : Marshal.dump(value)
          return true if serialized_value.size >= compress_threshold   
        end
      end
      false  
    end
    

    Note the assignment to serialized_value. If you poke around inside cache.rb, you'll see that it uses Marshal to serialize objects to byte strings before they go into the cache and then Marshal again to deserialize objects. The compression issue isn't important here, the important thing is the use of Marshal.

    The problem is that:

    Some objects cannot be dumped: if the objects to be dumped include bindings, procedure or method objects, instances of class IO, or singleton objects, a TypeError will be raised.

    Some things have state (such as OS file descriptors or blocks) that can't be serialized by Marshal. The error you're noting is this:

    can't dump hash with default proc

    So someone in your model has an instance variable that is a Hash and that Hash uses a block to supply default values. The column_methods_hash method uses such a Hash and even caches the Hash inside @dynamic_methods_hash; column_methods_hash will be called (indirectly) by public methods such as respond_to? and method_missing.

    One of respond_to? or method_missing will probably get called on every AR model instance sooner or later and calling either method makes your object unserializable. So, AR model instances are essentially unserializable in Rails 3.

    Interestingly enough, the respond_to? and method_missing implementations in 2.3.8 are also backed by a Hash that uses a block for default values. The 2.3.8 cache is "[...]is meant for caching strings." so you were getting lucky with a backend that could handle whole objects or it used Marshal before your objects had hash-with-procs in them; or perhaps you were using the MemoryStore cache backend and that's little more than a big Hash.

    Using multiple scope-with-lambdas might end up storing Procs in your AR objects; I'd expect the lambdas to be stored with the class (or singleton class) rather than the objects but I didn't bother with an analysis as the problem with respond_to? and method_missing makes the scope issue irrelevant.

    What You Can Do About It

    I think you've been storing the wrong things in your cache and getting lucky. You can either start using the Rails cache properly (i.e. store simple generated data rather than whole models) or you can implement the marshal_dump/marshal_load or _dump/_load methods as outlined in Marshal. Alternatively, you can use one of the MemoryStore backends and limit yourself to one distinct cache per server process.


    Executive Summary

    You can't depend on storing ActiveRecord model objects in the Rails cache unless you're prepared to handle the marshalling yourself or you want to limit yourself to the MemoryStore cache backends.


    The exact source of the problem has changed in more recent versions of Rails but there are still many instances of default_procs associated with Hashes.

    0 讨论(0)
  • 2020-11-29 05:10

    I realized that using where or some scope created ActiveRecord::Relation objects. I then noticed that doing a simple Model.find worked. I suspected that it didn't like the ActiveRecord::Relation object so I forced conversion to a plain Array and that worked for me.

    Rails.cache.fetch([self.id, 'relA']) do
      relA.where(
          attr1: 'some_value'
      ).order(
          'attr2 DESC'
      ).includes(
          :rel_1,
          :rel_2
      ).decorate.to_a
    end
    
    0 讨论(0)
  • 2020-11-29 05:15

    Thanks to mu-is-too-short for his excellent analysis. I've managed to get my model to serialize now with this:

    def marshal_dump
      {}.merge(attributes)
    end
    
    def marshal_load stuff
      send :initialize, stuff, :without_protection => true
    end
    

    I also have some "virtual attributes" set by a direct SQL join query using AS e.g. SELECT DISTINCT posts.*, name from authors AS author_name FROM posts INNER JOIN authors ON author.post_id = posts.id WHERE posts.id = 123. For these to work I need to declare an attr_accessor for each, then dump/load them too, like so:

    VIRTUAL_ATTRIBUTES = [:author_name]
    
    attr_accessor *VIRTUAL_ATTRIBUTES
    
    def marshal_dump
      virtual_attributes = Hash[VIRTUAL_ATTRIBUTES.map {|col| [col, self.send(col)] }]
      {}.with_indifferent_access.merge(attributes).merge(virtual_attributes)
    end
    
    def marshal_load stuff
      stuff = stuff.with_indifferent_access
      send :initialize, stuff, :without_protection => true
      VIRTUAL_ATTRIBUTES.each do |attribute|
        self.send("#{attribute}=", stuff[attribute])
      end
    end
    

    Using Rails 3.2.18

    0 讨论(0)
提交回复
热议问题