From b5327e432c59cdc77ec45e1a3594ccb5a147e1bc Mon Sep 17 00:00:00 2001 From: Raunaq Morarka Date: Sun, 10 Jul 2022 23:12:12 +0530 Subject: [PATCH] Avoid including page loading time in project expression profiler --- .../src/main/java/io/trino/operator/project/PageProcessor.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/core/trino-main/src/main/java/io/trino/operator/project/PageProcessor.java b/core/trino-main/src/main/java/io/trino/operator/project/PageProcessor.java index ed5cc98de9f3..d285836bdfd9 100644 --- a/core/trino-main/src/main/java/io/trino/operator/project/PageProcessor.java +++ b/core/trino-main/src/main/java/io/trino/operator/project/PageProcessor.java @@ -328,8 +328,9 @@ private ProcessBatchResult processBatch(int batchSize) } else { if (pageProjectWork == null) { + Page loadedPage = projection.getInputChannels().getInputChannels(page); expressionProfiler.start(); - pageProjectWork = projection.project(session, yieldSignal, projection.getInputChannels().getInputChannels(page), positionsBatch); + pageProjectWork = projection.project(session, yieldSignal, loadedPage, positionsBatch); expressionProfiler.stop(positionsBatch.size()); } if (!pageProjectWork.process()) {