Report Node_Name in analytics reports rather than country based on IP

Question

Report Node_Name in analytics reports rather than country based on IP

Closed this issue 12 years ago · 11 comments

The production Singapore node's IP (54.251.61.159) is being reported as being based in the US.

Rather than deduce the Country based on the IP as we are doing currently, can we rather just report the friendly Node_Name. Not only would this be simpler to implement / maintain, it would also be more accurate 😏

Answer 1 · 2012-09-11T04:51:59.000Z

It's not too complicated to do.
Alternatively, you can override this location in the GeoIP\override.txt file

Answer 2 · 2012-10-03T08:28:07.000Z

Added extra parameter for this - SliceByNodeName=yes

Answer 3 · 2012-10-04T04:29:19.000Z

PS this works for external tracker, but doesn't for built-in (because it sends data separately to a different session file)
Making it working for built-in will require refactoring of CIAPI.CS built-in tracker.

Answer 4 · 2012-10-04T09:32:48.000Z

Could you create an appropriate pull request for CIAPI.CS please.

Answer 5 · 2012-10-04T10:34:01.000Z

After some more analyzing - maybe it's better to combine data from the latency collector and CIAPI.CS into the same session file on server side. This needs some additional logic in the tracker, but makes analysis and comparison of data way much easier.
What you think?

Answer 6 · 2012-10-04T10:41:04.000Z

I think we should only have 1 set of data reported and stored. Having the events named "Login" rather than "/tradingapi/session" is more convienient for analysis.

I'll leave you to choose the best way to implement that (use one of the two existing trackers / combine into one etc.)

Answer 7 · 2012-10-04T10:52:50.000Z

I think it's better to keep 2 sets for now. One gives us network latency, and another network + CIAPI.CS (which is not null yet, at least for some methods). But we can give them different name prefixes.

Converting URLs to friendly names is useful (actually I already done it today). But I'm not sure how to implement it better, because it's specific for CIAPI.CS and not really a part of AppMetrics. Make it a plugin?

Answer 8 · 2012-10-04T11:11:21.000Z

PS the most simple way is to avoid using MetricsRecorder class and create custom recorder sending data using normal AppMetrics.Client

Answer 9 · 2012-10-04T12:03:36.000Z

@bitpusher - can you chime in on this one please

Answer 10 · 2012-10-24T12:46:50.000Z

wierd, i did not get a notification of this issue. we can change the metrics recorder any way you like

Answer 11 · 2013-01-24T05:40:50.000Z

Done. Used custom recorder class.