chore(pushbox): remove statsd metric tags

Because:
 - the statsd metric tags for pushbox are unnecessarily wasteful

This commit:
 - remove the tags
This commit is contained in:
Barry Chen 2022-11-08 16:21:35 -06:00 коммит произвёл dschom
Родитель 8efe4648a8
Коммит f94b9e9429
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: F26AEE99174EE68B
2 изменённых файлов: 3 добавлений и 8 удалений

Просмотреть файл

@ -176,11 +176,7 @@ export const pushboxApi = (
'pushbox.db.retrieve.success',
performance.now() - startTime
);
statsd.increment('pushbox.db.retrieve', {
uid,
deviceId,
msgCount: result.messages.length.toString(),
});
statsd.increment('pushbox.db.retrieve');
return {
last: result.last,
index: result.index,
@ -254,7 +250,7 @@ export const pushboxApi = (
'pushbox.db.store.success',
performance.now() - startTime
);
statsd.increment('pushbox.db.store', { uid, deviceId });
statsd.increment('pushbox.db.store');
return { index: result.idx };
} catch (err) {
statsd.timing(

Просмотреть файл

@ -97,8 +97,7 @@ describe('pushbox', () => {
);
sinon.assert.calledOnceWithExactly(
mockStatsD.increment,
'pushbox.db.store',
{ uid: mockUid, deviceId: mockDeviceIds[0] }
'pushbox.db.store'
);
assert.equal(index, '12');
});