[jitsi-dev] [jitsi-videobridge] upgrade influxdb to 0.9.2 (#80)


#1

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

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

-- Commit Summary --

  * upgrade influxdb to 0.9.2

-- File Changes --

    M src/main/java/org/jitsi/videobridge/influxdb/LoggingHandler.java (147)

-- Patch Links --

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

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/80


#2

@@ -86,27 +86,27 @@
             "conference_id",
             "endpoint_id",

- "stats.local_ip",
- "stats.local_port",
- "stats.remote_ip",
- "stats.remote_port",
+ "stats_local_ip",

0.9.2 has a bug where it doesn't accept column names that start with reserved words. (https://github.com/influxdb/influxdb/issues/3547)

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/80/files#r36992506


#3

- Object[] values = e.getValues();
-
- if (useLocalTime)
- columns.add("time");
- Collections.addAll(columns, e.getColumns());
-
- if (values[0] instanceof Object[])
- {
- multipoint = true;
- pointCount = values.length;
- }
-
- if (multipoint)
- {
- for (int i = 0; i < pointCount; i++)
+ /* The following is a sample JSON message in the format used by InfluxDB v0.9.0

Technically, this JSON format is deprecated as of 0.9.2, so next time we upgrade, we'll likely have to switch to their new custom text-based format for data submissions.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/80/files#r36992612


#4

Can one of the admins verify this patch?

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/80#issuecomment-192016169


#5

Hi, thanks for your contribution!
If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

···

---
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/80#issuecomment-218796979


#6

Closed #80.

···

---
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/80#event-658881536