From b8d6ecbb8925d3d13d62ef51814a629459b8b327 Mon Sep 17 00:00:00 2001 From: John Lees-Miller Date: Wed, 2 Aug 2023 21:07:13 +0100 Subject: [PATCH] Merge pull request #14143 from overleaf/jlm-unsampled-compiles Remove compile sampling for analysis GitOrigin-RevId: 2457977d61756be52db506d7c84531b22a26abe0 --- .../web/app/src/Features/Compile/CompileController.js | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/services/web/app/src/Features/Compile/CompileController.js b/services/web/app/src/Features/Compile/CompileController.js index c5fd9b64b2..57481389ab 100644 --- a/services/web/app/src/Features/Compile/CompileController.js +++ b/services/web/app/src/Features/Compile/CompileController.js @@ -164,14 +164,7 @@ module.exports = CompileController = { pdfDownloadDomain += outputUrlPrefix } - if ( - limits && - SplitTestHandler.getPercentile( - AnalyticsManager.getIdsFromSession(req.session).analyticsId, - 'compile-result-backend', - 'release' - ) === 1 - ) { + if (limits) { // For a compile request to be sent to clsi we need limits. // If we get here without having the limits object populated, it is // a reasonable assumption to make that nothing was compiled.