[jitsi-dev] [jitsi/jitsi-videobridge] Adds callstats user statistics per endpoint (#231)


#1

You can view, comment on, or merge this pull request online at:

  https://github.com/jitsi/jitsi-videobridge/pull/231

-- Commit Summary --

  * Updates to latest callstats 20160426.085453-7.
  * Make some constant fields public, to be accessable from other packages.
  * Register CallStats as a service to be accessed from other bundles.
  * Adds CallStats bundle in eventadmin package, that sends client statistics per conference, per channel, per ssrc.

-- File Changes --

    M pom.xml (2)
    A src/main/java/org/jitsi/videobridge/eventadmin/callstats/Activator.java (151)
    A src/main/java/org/jitsi/videobridge/eventadmin/callstats/CallStatsConferenceStatsHandler.java (358)
    M src/main/java/org/jitsi/videobridge/osgi/JvbBundleConfig.java (3)
    M src/main/java/org/jitsi/videobridge/stats/CallStatsIOTransport.java (16)
    M src/main/java/org/jitsi/videobridge/stats/StatsManagerBundleActivator.java (6)

-- Patch Links --

https://github.com/jitsi/jitsi-videobridge/pull/231.patch
https://github.com/jitsi/jitsi-videobridge/pull/231.diff

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/231


#2

Depends on https://github.com/jitsi/libjitsi/pull/154. And needs it first to be merged and then this one needs an update in pom.xml with the correct libjitsi version.

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/231#issuecomment-215447789


#3

+ (CallStats) service,
+ bridgeId,
+ interval);
+
+ String[] topics = { "org/jitsi/*" };
+
+ serviceRegistration
+ = EventUtil.registerEventHandler(
+ bundleContext,
+ topics,
+ conferenceStatsHandler);
+ }
+ }
+ break;
+
+ case ServiceEvent.UNREGISTERING:

@damencho, shouldn't you be listening to UNREGISTERING of CallStats only here and not just any OSGi service? Like you do in REGISTERED?

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/231/files/d38928a8bae4aa24309d86391b57f5fa6f0c19e3#r63449996


#4

Merged #231.

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/231#event-662535274