Skip to content

Commit 477108a

Browse files
committed
TASK: Add rendertime to render orchestrator output
1 parent a0de60d commit 477108a

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

Classes/NodeRendering/NodeRenderOrchestrator.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,8 @@ public function renderContentRelease(ContentReleaseIdentifier $contentReleaseIde
122122
return;
123123
}
124124

125+
$startTime = time();
126+
125127
// Ensure we start with an empty queue here, in case this command is called multiple times.
126128
$this->redisRenderingQueue->flush($contentReleaseIdentifier);
127129
$this->redisRenderingErrorManager->flush($contentReleaseIdentifier);
@@ -172,7 +174,7 @@ public function renderContentRelease(ContentReleaseIdentifier $contentReleaseIde
172174

173175
if (count($nodesScheduledForRendering) === 0) {
174176
// we have NO nodes scheduled for rendering anymore, so that means we FINISHED successfully.
175-
$contentReleaseLogger->info('Everything rendered completely. Finishing RenderOrchestrator');
177+
$contentReleaseLogger->info(sprintf('Everything rendered completely in %d seconds. Finishing RenderOrchestrator', time() - $startTime));
176178

177179
// info to all renderers that we finished, and they should terminate themselves gracefully.
178180
$this->redisContentReleaseService->setContentReleaseMetadata($contentReleaseIdentifier, $releaseMetadata->withStatus(NodeRenderingCompletionStatus::success())->withEndTime(new \DateTimeImmutable()), RedisInstanceIdentifier::primary());

0 commit comments

Comments
 (0)