From d412ca93d3a789f0cc46ce148bfaa02aed61e8ea Mon Sep 17 00:00:00 2001
From: Arash Deshmeh <adshmh@gmail.com>
Date: Sat, 17 Sep 2022 18:32:25 -0400
Subject: [PATCH] Staging: Remove metrics recording from merge check path

Signed-off-by: Arash Deshmeh <adshmh@gmail.com>
---
 src/services/pocket-relayer.ts | 32 --------------------------------
 1 file changed, 32 deletions(-)

diff --git a/src/services/pocket-relayer.ts b/src/services/pocket-relayer.ts
index 49f7d80e..6d532792 100644
--- a/src/services/pocket-relayer.ts
+++ b/src/services/pocket-relayer.ts
@@ -709,38 +709,6 @@ export class PocketRelayer {
       if (mergeCheckResult) {
         nodes = mergeCheckResult.nodes
       } else {
-        const error = 'Merge check failure: '
-
-        const method = 'checks'
-
-        this.metricsRecorder
-          .recordMetric({
-            requestID,
-            applicationID,
-            applicationPublicKey,
-            blockchainID,
-            serviceNode: 'session-failure',
-            relayStart,
-            result: 500,
-            bytes: Buffer.byteLength(error, 'utf8'),
-            fallback: false,
-            method,
-            error,
-            code: undefined,
-            origin: this.origin,
-            data,
-            session: this.session,
-            gigastakeAppID: applicationID !== application.id ? application.id : undefined,
-          })
-          .catch(function log(e) {
-            logger.log('error', 'Error recording metrics: ' + e, {
-              requestID,
-              relayType: 'APP',
-              typeID: application.id,
-              serviceNode: 'session-failure',
-            })
-          })
-
         return new Error('Merge check failure; using fallbacks')
       }
     }