Default worker exception handler should not exit the quebert worker loop

[bradgessler]

The current default behavior of the quebert worker loop is to exit the quebert process when an exception is raised:

https://github.com/polleverywhere/quebert/blob/master/lib/quebert/worker.rb#L33

Change the default to log the exception and bury the job for later inspection. Also make the object passed into the error proc a custom struct/class for better error encapsulation.

• Change the default behavior at https://github.com/polleverywhere/quebert/blob/master/lib/quebert/worker.rb#L33 to log the error to Quebert.logger.error and bury the job.
• Create a Quebert::Worker::Error object that's passed into the exception handler proc via https://github.com/polleverywhere/quebert/blob/master/lib/quebert/worker.rb#L26.
  • #to_s method returns all of the info needed for the logger.error call (e.g. Quebert.logger.error(worker_error)

Since this breaks the 3.0 error handler API this should belong on Quebert 4.0.

[bradgessler]

We should consider moving as much error handling, etc. into the job as we can. I think this could eliminate the need for all the different controllers we have and most of our hooks if we can pull this off.