Comments (5)
@stanislas as a user of the tool do you have a point of view?
from jvm-breakglass.
I think that the logging is because the numbers of logged event is quite small: at startup and when starting/stoping in between. Also, the "normal" messages are logged at INFO Level, so it is not intrusive.
from jvm-breakglass.
Ok it makes sense. However during the test the number of exceptions logged is quite high even though the tests pass. I think from a testing perspective it can be quite confusing to follow the logs when you see multiple exceptions and stack traces being displayed.
I propose to pass an additional boolean in the NreplServer constructor to make exception logs silent when required. I could use this boolean in during the test to make the log more readable. However the default behaviour i.e. the constuctor with one parameter (port number) would keep logging exception as you suggest (to keep the current behaviour).
I can do that.
Any reserve?
from jvm-breakglass.
Done.
from jvm-breakglass.
it's good for me ;)
from jvm-breakglass.
Related Issues (13)
- Write a test to validate the thread safety of start/stop HOT 1
- Use travis-ci.org for Continuous Integration HOT 3
- Build on Travis fails from time to time
- Writing java doc
- Update Readme with specific examples on how to setup the JMX switch
- lein repl :connect localhost:1112 doesn't work HOT 5
- Maven repo? HOT 1
- Update docs to recommend require
- Use java.util.logging.Logger instead of out.println HOT 2
- Catching exception or not catching the exception? That is the question. HOT 3
- Writing test to cover the initialisation of the MBean + start stop functionality HOT 4
- Write test to explicitly load a test classcontext to test NreplServerSpring HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from jvm-breakglass.