-
Notifications
You must be signed in to change notification settings - Fork 105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feature/320_die_if_no_logs #337
Conversation
frbattid
commented
Mar 5, 2015
- Implements issue https://github.com/telefonicaid/fiware-connectors/issues/320
- Fixes issue https://github.com/telefonicaid/fiware-connectors/issues/302
- 100% unit tests passed
- (Unofficial) e2e tests passed (emulating a persistence error, ttl=2)
- Ths PR must be twined into develop
- Assignee: @fgalan , but LGTM from @gtorodelvalle is also required
info("[" + componentName + "] " + message); | ||
} catch (Exception e) { | ||
if (shutdown) { | ||
System.err.println("A problem with the logging system was found... shutting down Cygnus right now!"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe you should also print the Exception trace/information to provide more information to the administrator reading the mesage?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for commenting on the commits :( I left-click-opened them and start commenting there :( I will comment on the pull request "File changed" tab from now on ;) Anyhow, I think all my comments are included so LGTM on my side ;) Thanks @frbattid ;) |
Conflicts: flume/CHANGES_NEXT_RELEASE flume/src/main/java/es/tid/fiware/fiwareconnectors/cygnus/sinks/OrionSink.java
Please ignore the following commits:
|
Conflicts: flume/CHANGES_NEXT_RELEASE flume/src/main/java/es/tid/fiware/fiwareconnectors/cygnus/nodes/CygnusApplication.java
LGTM (Although @gtorodelvalle also told LGTM, not sure if I should merge, given the 8 commits since that point. If you want me to merge, just tell me). |
feature/320_die_if_no_logs
feature/320_die_if_no_logs