Initial work for sending events on request success only.

Use a EventSink singleton, but with ThreadLocal list of events to send.

A servlet filter kicks in to make sure we dispatch events.
Send events after successful pinsetter jobs.
Request scoped event sink, and introducing pinsetter guice module.
Add a PinsetterJobScope.
Seed PinsetterScope with EventSink.
Consolidate Guice modules.
Create an EventSinkProvider that handles scoping correctly.
Dispatch events on successful requests.

Mistakenly assumed these bubble up as exceptions here when infact we just get a
normal response object. Cast similar to LoggingFilter, get the status, and use
this to determine if the request was a success or not.
Unit test fixes.
Fix job factory unit test.

Removing the class comparison in the test, it looks of very little significance
and this would be at least the second time it's needed to be updated. Too
fragile for a complex area of the code where we need to wrap/decorate, we
shouldn't care what specific class is returned.
Check if both request and pinsetter sinks are non-null and error out.
Minor cleanup.
Fix more issues with sending events on successful requests.
Checkstyle.
Comment updates.
Fix import ordering and unnecessary log debug guards.
Return to injecting EventSink normally in ConsumerResource.
Improve logic for determining if request is successful.
Correct import ordering.
Merge pull request #719 from candlepin/dgoodwin/events-on-success

Dgoodwin/events on success