On Tue, Oct 05, 2010 at 06:34:38AM +0000, Hartog C. de Mik wrote: > On Tue, Oct 05, 2010 at 08:54:30AM +0900, John Mettraux wrote: > > > > I had a closer look at your pull request. > > > > http://github.com/kennethkalmer/ruote-amqp/pull/1 > > > > It's not necessary. Informations like "command" and "reply_queue" are > > already passed in > > > > workitem['params']['participant_options'] > > > > > > Thanks anyway, kind regards, > > My version of daemon-kit > > % daemon-kit --version > daemon-kit 0.1.8.1 (http://github.com/kennethkalmer/daemon-kit) > > does not listen to 'command' and 'reply_queue' from the participant > options, but seeks them in the first level of the workitem hash.
Hello Hartog, I don't understand, what has daemon-kit to do with that ? The participant implimentation is at http://github.com/jmettraux/ruote-amqp/blob/ruote2.1/lib/ruote-amqp/participant.rb which is not part of daemon-kit. > In one of your comments of my investigation into ruote and amqp you > noted that the ":command => '/foo/foo'" and ":reply_queue => > 'ruote_workitems'" have no place in the process definition and I fully > agree. > > But with the current code base this is just not possible - command and > reply_queue must be specified within the process definition > > The pull I requested would fix just that. (And set the default reply > queue which seems to make sense) What do you mean by current code base ? If you read carefully http://ruote.rubyforge.org/source.html you'll see that the latest version of ruote-amqp is at : http://github.com/jmettraux/ruote-amqp Kenneth has not had time to work on anything ruote related since february. Torsten and I have taken over. The latest version packages the command and reply queue info into workitem.fields['params']['participant_options']. If not, please provide a spec that proves it does not. Cheers, -- John Mettraux - http://jmettraux.wordpress.com -- 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
