fix: Add queryId to query failed event (#25476)

PostHog · Oct 9, 2024 · cf39f4b · cf39f4b
1 parent 85f9317
commit cf39f4b
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/frontend/src/queries/query.ts b/frontend/src/queries/query.ts
@@ -147,10 +147,20 @@ export async function performQuery<N extends DataNode>(
                 logParams.clickhouse_sql = (response as HogQLQueryResponse)?.clickhouse
             }
         }
-        posthog.capture('query completed', { query: queryNode, duration: performance.now() - startTime, ...logParams })
+        posthog.capture('query completed', {
+            query: queryNode,
+            queryId,
+            duration: performance.now() - startTime,
+            ...logParams,
+        })
         return response
     } catch (e) {
-        posthog.capture('query failed', { query: queryNode, duration: performance.now() - startTime, ...logParams })
+        posthog.capture('query failed', {
+            query: queryNode,
+            queryId,
+            duration: performance.now() - startTime,
+            ...logParams,
+        })
         throw e
     }
 }