[GitHub] 1u0 commented on issue #6976: [FLINK-7155][metrics] Add new metrics reporter to InfluxDB

2019-01-28 Thread GitBox
1u0 commented on issue #6976: [FLINK-7155][metrics] Add new metrics reporter to InfluxDB URL: https://github.com/apache/flink/pull/6976#issuecomment-458236341 @knaufk, this may be due to some bug in (older) Flink runtime. I don't see the index value in the resulting name, by running

[GitHub] 1u0 commented on issue #6976: [FLINK-7155][metrics] Add new metrics reporter to InfluxDB

2018-11-07 Thread GitBox
1u0 commented on issue #6976: [FLINK-7155][metrics] Add new metrics reporter to InfluxDB URL: https://github.com/apache/flink/pull/6976#issuecomment-436570526 @zentol, an exception would be thrown on `report()`. What do you think about making `db` configuration parameter as

[GitHub] 1u0 commented on issue #6976: [FLINK-7155][metrics] Add new metrics reporter to InfluxDB

2018-11-07 Thread GitBox
1u0 commented on issue #6976: [FLINK-7155][metrics] Add new metrics reporter to InfluxDB URL: https://github.com/apache/flink/pull/6976#issuecomment-436566905 @rmetzger > Will the influxdb client just throw an exception if the db doesn't exist? Yes, if the database doesn't

[GitHub] 1u0 commented on issue #6976: [FLINK-7155][metrics] Add new metrics reporter to InfluxDB

2018-10-31 Thread GitBox
1u0 commented on issue #6976: [FLINK-7155][metrics] Add new metrics reporter to InfluxDB URL: https://github.com/apache/flink/pull/6976#issuecomment-434679481 Notes: * this PR is an alternative of #4299, but uses different InfluxDB client library. Also, it's (maybe) not directly