remove perf logging

Author: KSDaemon

packages/cubejs-schema-compiler/src/compiler/DataSchemaCompiler.ts

@@ -227,8 +227,6 @@ export class DataSchemaCompiler {
       ? files.filter(f => this.filesToCompile.includes(f.fileName)).map(f => ({ ...f }))
       : files.map(f => ({ ...f }));
 
-    const jinjaLoaderTimer = perfTracker.start('loadJinjaTemplates');
-
     const jinjaTemplatedFiles = toCompile.filter((file) => file.fileName.endsWith('.jinja') ||
       (file.fileName.endsWith('.yml') || file.fileName.endsWith('.yaml')) && file.content.match(JINJA_SYNTAX));
 
@@ -237,8 +235,6 @@ export class DataSchemaCompiler {
       this.loadJinjaTemplates(jinjaTemplatedFiles);
     }
 
-    jinjaLoaderTimer.end();
-
     const errorsReport = new ErrorReporter(null, [], this.errorReportOptions);
     this.errorsReporter = errorsReport;
 
@@ -566,8 +562,6 @@ export class DataSchemaCompiler {
   ): Promise<(FileContent | undefined)> {
     try {
       if (getEnv('transpilationNative')) {
-        const compileJsFileTimer = perfTracker.start('transpileJsFile (native)');
-
         const reqData = {
           fileName: file.fileName,
           fileContent: file.content,
@@ -589,12 +583,8 @@ export class DataSchemaCompiler {
         errorsReport.addWarnings(res[0].warnings as unknown as SyntaxErrorInterface[]);
         errorsReport.exitFile();
 
-        compileJsFileTimer.end();
-
         return { ...file, content: res[0].code };
       } else if (getEnv('transpilationWorkerThreads')) {
-        const compileJsFileTimer = perfTracker.start('transpileJsFile (threads)');
-
         const data = {
           fileName: file.fileName,
           content: file.content,
@@ -607,12 +597,8 @@ export class DataSchemaCompiler {
         errorsReport.addErrors(res.errors);
         errorsReport.addWarnings(res.warnings);
 
-        compileJsFileTimer.end();
-
         return { ...file, content: res.content };
       } else {
-        const compileJsFileTimer = perfTracker.start('transpileJsFile (inplace)');
-
         const ast = parse(
           file.content,
           {
@@ -630,8 +616,6 @@ export class DataSchemaCompiler {
 
         const content = babelGenerator(ast, {}, file.content).code;
 
-        compileJsFileTimer.end();
-
         return { ...file, content };
       }
     } catch (e: any) {
@@ -665,8 +649,6 @@ export class DataSchemaCompiler {
     }
 
     if (getEnv('transpilationNative')) {
-      const transpileYamlFileTimer = perfTracker.start('transpileYamlFile (native)');
-
       const reqData = {
         fileName: file.fileName,
         fileContent: file.content,
@@ -685,12 +667,8 @@ export class DataSchemaCompiler {
 
       this.compiledYamlCache.set(cacheKey, res[0].code);
 
-      transpileYamlFileTimer.end();
-
       return { ...file };
     } else if (getEnv('transpilationWorkerThreads')) {
-      const transpileYamlFileTimer = perfTracker.start('transpileYamlFile (threads)');
-
       const data = {
         fileName: file.fileName,
         content: file.content,
@@ -708,12 +686,8 @@ export class DataSchemaCompiler {
 
       this.compiledYamlCache.set(cacheKey, res.content);
 
-      transpileYamlFileTimer.end();
-
       return { ...file };
     } else {
-      const transpileYamlFileTimer = perfTracker.start('transpileYamlFile (inplace)');
-
       const transpiledFile = this.yamlCompiler.transpileYamlFile(file, errorsReport);
 
       if (transpiledFile) {
@@ -723,8 +697,6 @@ export class DataSchemaCompiler {
 
       this.compiledYamlCache.set(cacheKey, transpiledFile?.content || '');
 
-      transpileYamlFileTimer.end();
-
       return transpiledFile;
     }
   }
@@ -734,8 +706,6 @@ export class DataSchemaCompiler {
     errorsReport: ErrorReporter,
     options: TranspileOptions
   ): Promise<(FileContent | undefined)> {
-    const transpileJinjaFileTimer = perfTracker.start('transpileJinjaFile (common)');
-
     const renderedFile = await this.yamlCompiler.renderTemplate(
       file,
       this.standalone ? {} : this.cloneCompileContextWithGetterAlias(this.compileContext),
@@ -748,8 +718,6 @@ export class DataSchemaCompiler {
     // avoiding costly YAML/Python parsing again.
     file.content = renderedFile.content;
 
-    transpileJinjaFileTimer.end();
-
     return this.transpileYamlFile(file, errorsReport, options);
   }
 

packages/cubejs-schema-compiler/src/compiler/YamlCompiler.ts

@@ -19,7 +19,6 @@ import { nonStringFields } from './CubeValidator';
 import { ErrorReporter } from './ErrorReporter';
 import { camelizeCube } from './utils';
 import { CompileContext } from './DataSchemaCompiler';
-import { perfTracker } from './PerfTracker';
 
 type EscapeStateStack = {
   inFormattedStr?: boolean;