alois/Verso
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[web] bring back compileTimeServerE2E in client side compile metrics (#9203)

Browse Source 
GitOrigin-RevId: f10ee837869b102aab25695c676d0e6adafa80c1
This commit is contained in:
Jakob Ackermann
2022-08-11 10:19:26 +01:00
committed by Copybot
parent e103f736e1
commit f2b0b77907
1 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
services/web/frontend/js/features/pdf-preview/util/metrics.js
+3 -7
View File
@@ -4,7 +4,7 @@ import { trackPdfDownloadEnabled } from './pdf-caching-flags'
// VERSION should get incremented when making changes to caching behavior or // VERSION should get incremented when making changes to caching behavior or
// adjusting metrics collection. // adjusting metrics collection.
const VERSION = 5 const VERSION = 6
// editing session id // editing session id
const EDITOR_SESSION_ID = uuid() const EDITOR_SESSION_ID = uuid()
@@ -33,8 +33,7 @@ export function trackPdfDownload(response, compileTimeClientE2E, t0) {
if (!isFirstRender) return if (!isFirstRender) return
isFirstRender = false isFirstRender = false
const totalDeliveryTime = Math.ceil(performance.now() - t0) deliveryLatencies.totalDeliveryTime = Math.ceil(performance.now() - t0)
deliveryLatencies.totalDeliveryTime = totalDeliveryTime
deliveryLatencies.latencyFetch = latencyFetch deliveryLatencies.latencyFetch = latencyFetch
if (latencyRender) { if (latencyRender) {
deliveryLatencies.latencyRender = latencyRender deliveryLatencies.latencyRender = latencyRender
@@ -42,10 +41,7 @@ export function trackPdfDownload(response, compileTimeClientE2E, t0) {
if (trackPdfDownloadEnabled) { if (trackPdfDownloadEnabled) {
// Submit latency along with compile context. // Submit latency along with compile context.
submitCompileMetrics({ submitCompileMetrics({
totalDeliveryTime, ...deliveryLatencies,
latencyFetch,
latencyRender,
compileTimeClientE2E,
...pdfCachingMetrics, ...pdfCachingMetrics,
}) })
} }