Catrope has uploaded a new change for review.
https://gerrit.wikimedia.org/r/178415
Change subject: [WIP] Move from legacy TimingData schema to new statsd API
......................................................................
[WIP] Move from legacy TimingData schema to new statsd API
This API doesn't have the ability to track any other data
in addition to a name and a duration in milliseconds, so
we add the target name ('mwTarget' or 'mobile') to the
event name and discard all other data.
Change-Id: I25ae0243f8983142d7755b65b05c18d5df36a253
---
M modules/ve-mw/init/ve.init.mw.trackSubscriber.js
1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor
refs/changes/15/178415/1
diff --git a/modules/ve-mw/init/ve.init.mw.trackSubscriber.js
b/modules/ve-mw/init/ve.init.mw.trackSubscriber.js
index 0906a7d..16c3072 100644
--- a/modules/ve-mw/init/ve.init.mw.trackSubscriber.js
+++ b/modules/ve-mw/init/ve.init.mw.trackSubscriber.js
@@ -58,9 +58,9 @@
prefix = topic.slice( 0, topic.indexOf( '.' ) );
if ( prefix === 'mwtiming' ) {
- // Legacy TimingData events
- // Map timing.foo --> ve.foo
- topic = 've.' + topic.slice( prefix.length + 1 );
+ // Map mwtiming.foo --> timing.mobile.foo
+ topic = 'timing.' + data.targetName + '.' +
topic.slice( 'mwtiming.'.length );
+ data = data.duration;
} else if ( prefix === 'mwedit' ) {
// Edit schema
action = topic.split( '.' )[1];
--
To view, visit https://gerrit.wikimedia.org/r/178415
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I25ae0243f8983142d7755b65b05c18d5df36a253
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
