sekharnbos / activemessaging

Automatically exported from code.google.com/p/activemessaging
0 stars 0 forks source link

shouldn't swallow exceptions when trying to load messaging.rb #13

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?

 syntax error in messaging.rb, or just rename it

What is the expected output? What do you see instead?

 a13g bombs when trying to subscribe to queues that aren't loaded

What version of the product are you using? On what operating system?

 rev 100

Original issue reported on code.google.com by dennis.b...@gmail.com on 3 Jun 2007 at 11:42

Attachments:

GoogleCodeExporter commented 8 years ago
Not sure I understand - don't you want the system to fail when you have a bad
configuration?

For example, rails does not just keep going when you have a bad configuration 
(e.g.
most of the database and env params, if wrong, cause rails to not even start - 
these
config errors are not swallowed).

Please make more of a case as to why the app should be allowed to continue with 
this
kind of error - I believe the intent of the design was to not allow these 
errors to
be ignored.

Original comment by kooks...@gmail.com on 4 Jun 2007 at 1:56

GoogleCodeExporter commented 8 years ago
ok - now that I look at your patch more, I understand what you mean - 
I think the reason for not throwing an error on that was that it is perfectly 
valid
to have the activemessgaing plugin installed, but not have that messaging.rb. 
So, for
that reason, I'll have it be a bit more loud, but not fail if the file is not 
there.

Original comment by kooks...@gmail.com on 6 Jul 2007 at 9:40

GoogleCodeExporter commented 8 years ago

Original comment by kooks...@gmail.com on 13 Jul 2007 at 12:51