Closed pmiddend closed 5 years ago
@wsmirnow @staubesv I'm assuming that Plapadoo and co should have write access to this repo. I've given @doofy and @pmiddend write for this repo. Anyone else needed?
Hi Greg, you can grant write permissions to all Opencast developers. But it will take a while. Doing it on demand is also an option. No really, I havn't a strong opinion about that. It is ok, if the main developer have write access to this repo.
Regards Waldemar
ELAN e.V. Heger-Tor-Wall 12 49074 Osnabrück - Germany Tel: +49 541 / 969 6533 Web: https://elan-ev.de
Am 03.05.19 um 17:51 schrieb Greg Logan:
@wsmirnow https://github.com/wsmirnow @staubesv https://github.com/staubesv I'm assuming that Plapadoo and co should have write access to this repo. I've given @doofy https://github.com/doofy and @pmiddend https://github.com/pmiddend write for this repo. Anyone else needed?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/opencast/opencast-influxdb-adapter/pull/1#issuecomment-489143632, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAGQ4CDEA3JCOLVAFAZGA3PTRNQPANCNFSM4HE3WWKQ.
@gregorydlogan I would say that all Opencast committers should have write access to this repo.
@staubesv @wsmirnow committers already have write access, but @pmiddend didn't. @doofy already did, but I forgot he was a committer :)
@wsmirnow What is the status of this review?
Looks fine for me. I havn't run this code but the build was successfull.
Thanks for the review!
MH-13483 introduces basic statistics visualization facilities into Opencast. In particular, a statistics provider implementation for InfluxDB 1.x time series is provided, i.e. Opencast can visualize InfluxDB time series data.
The Opencast InfluxDB Adapter (this pull request) is implemented as a standalone Java application that continuously parses the Apache log files HTTP access logs to gather data about the usage of assets (here: "Views" of video files) and feeds those info into InfluxDB.
The adapter optionally uses the External API to find out to which series a given event belongs so that view data can be aggregated on series level, too.
Currently implemented feature set:
Current limitations:
This work is sponsored by SWITCH.