Skip to content

Commit ac34539

Browse files
committed
remove perf logging
1 parent afcb21d commit ac34539

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

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

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -515,8 +515,6 @@ export class DataSchemaCompiler {
515515
errorsReport: ErrorReporter,
516516
{ cubeNames, cubeSymbols, contextSymbols, transpilerNames, compilerId, stage }: TranspileOptions
517517
): Promise<(FileContent | undefined)[]> {
518-
const compileJsFileTimer = perfTracker.start('transpileJsFilesBulk (native)');
519-
520518
// for bulk processing this data may be optimized even more by passing transpilerNames, compilerId only once for a bulk
521519
// but this requires more complex logic to be implemented in the native side.
522520
// And comparing to the file content sizes, a few bytes of JSON data is not a big deal here
@@ -536,8 +534,6 @@ export class DataSchemaCompiler {
536534
}));
537535
const res = await transpileJs(reqDataArr);
538536

539-
compileJsFileTimer.end();
540-
541537
return files.map((file, index) => {
542538
errorsReport.inFile(file);
543539
if (!res[index]) { // This should not happen in theory but just to be safe
@@ -747,9 +743,7 @@ export class DataSchemaCompiler {
747743
compiledFiles[file.fileName] = true;
748744

749745
if (file.convertedToJs) {
750-
const compileJsFileTimer = perfTracker.start('compileJsFile (convertedToJs)');
751746
this.compileJsFile(file, errorsReport);
752-
compileJsFileTimer.end();
753747
} else if (file.fileName.endsWith('.js')) {
754748
this.compileJsFile(file, errorsReport, { doSyntaxCheck });
755749
} else if (file.fileName.endsWith('.yml.jinja') || file.fileName.endsWith('.yaml.jinja') ||

0 commit comments

Comments
 (0)