Sqlite cleanup + correction

roq-messaging / RoQ

RoQ, Elastically scalable MOM

21 stars 4 forks source link

Sqlite cleanup + correction #165

Closed vanmellebenjamin closed 9 years ago

vanmellebenjamin commented 9 years ago

@sskhiri @ahderman @jbruggem

I cleaned the code by removing all the useless SQL classes and error handlers.

I discovered that a BSON_REQUEST (remove_queue) didn't send Ok reply. That led to a loop in the unit tests. The test wait for controller answers up to infinite (maybe we should change this behaviour). Now the tests pass.

sskhiri commented 9 years ago

Well done! I got into this kind of bugs so many times and it is quite hard to identify them quickly ... But it seems that the build is still failing, isn't it ?

vanmellebenjamin commented 9 years ago

For my own, the maven build pass always when tests are skipped. When you make a complete build with all the tests this is more random. Sometimes a JVM crashes happening, I will try again this afternoon and report the error trace.

ahderman commented 9 years ago

Can you please also check if some dependencies can be removed from the roq-management pom.xml?