Guillaume Lederrey
Guillaume Lederrey
Sorry for not reacting before. This looks good! I need to find some quite time to setup the correct variables and merge this... Thanks a lot!
Thanks a lot ! This looks good ! The aggregated reports is also something I wanted to do for a long time, but never took the time to actually do...
General question: do you think it is a good idea to extract `GIT_AUTHOR_NAME` and other such variables from the last git commit (which should identify the person who merged the...
@zonArt If you still have too much time at night, this PR would welcome some of your love...
Side note: you could move the outputwriters definitions to the server and not duplicate them for each query.
This PR is missing a unit test to demonstrate the feature. I'll try to have a go at it, but I am giving a training this week, so it will...
Very much possible that something is broken with InfluxDBWriter. Did you try another writer to check if the issue is output or input? For example the StdOutWriter ?
The [image on docker hub](https://hub.docker.com/r/jmxtrans/jmxtrans/) should be 260 (see [Dockerfile](https://github.com/jmxtrans/jmxtrans/blob/master/jmxtrans/docker/Dockerfile#L59), this is hardcoded at the moment, yes it is ugly, feel free to send a patch). I know mostly nothing...
A few comments... At the moment, we target Java 7. We should probably raise this to Java 8 at some point. I have no idea how many people still use...
Thanks for sending this fix! Any chance you could send it as a pull request and not an attached .zip file? It would be much easier to read and process!...