On Wed, Aug 03, 2011 at 08:08:35AM -0700, Akeem A. wrote:
>
>   Recently, I have attempted to use Ruote's CompositeStorage in my
> application with Ruote::MongoDbStorage used for history storage and
> Ruote::Redis::RedisStorage to store the rest of the the data. The
> exception received mentioned that CompositeStorage does not have an
> add_type method, my uncommenting the method resolves the issue and it
> looks like it should be there, but was there a historical reason for
> its initial removal?
>
> I made a pull request with the change on github: 
> https://github.com/jmettraux/ruote/pull/36

Hello Akeem,

thanks for your help. I have merged your pull request in. I have also added 
some rdoc to the method:

  
https://github.com/jmettraux/ruote/commit/977427b82f842c337d5f6d63f854bf7750594607

I had commented the method out since I didn't know to which underlying storage 
the new type should go. Since it works OK with an empty add_type, then let's go 
that way.

If problems arise, we should probably think about adding the new type to the 
default storage in the CompositeStorage. If I remember correclty this method 
only matters for the HashStorage that comes with ruote (in-memory storge).


Thanks again,

--
John Mettraux - http://lambda.io/processi

-- 
you received this message because you are subscribed to the "ruote users" group.
to post : send email to [email protected]
to unsubscribe : send email to [email protected]
more options : http://groups.google.com/group/openwferu-users?hl=en

Reply via email to