runa-dev/recognizer — comment on commit 7c515e7fa05a745ddb1ba3f32a5227426a2710ee
File: lib/recognizer/amqp.rb line 17
By abhijith:

On Sat, Feb 18, 2012 at 2:46 AM, Sean Porter

wrote:
> Although default values are great. Do you intend to put together a pull request? Did you run into any issues w/ recognizer?

Yeah, we did hit a few problems. One problem that we had was that
"message[:routing_key]" seems to be empty. Had to change it to
"message[:delivery_details][:routing_key]".
Is it working for you _without_ the change? If It is, then its
probably a rabbitmq version incompatibility. We are using rabbitmq 2.7
and I haven't been able to check with 2.4. If there is a change in the
message format, we'll need to accommodate that too. If this fix works
for both versions, then we are good to go and I can send you a pull
request.

The other problem was that graphite messages/lines were not being
picked up from the thread queue since the line was being split on
space and not tabs. Patched the code to split the line based on tabs
and spaces ([\s]), so that both work.

I think there was one more problem which I am not able to recollect at
the moment. Will take a look the patch and try to remember what it was

--
Abhi