Log number of live query runner instances

findepi · findepi · commit 93d0d037ac7a · 2024-03-15T10:16:31.000+01:00
On CI we observe that sometimes tests create much higher than expected
number of query runner instances and then often fail (too many resources
taken or too many tests running in parallel). Logging this number helps
attribute CI failure to this cause.
diff --git a/testing/trino-testing/src/main/java/io/trino/testing/DistributedQueryRunner.java b/testing/trino-testing/src/main/java/io/trino/testing/DistributedQueryRunner.java
@@ -218,7 +218,7 @@ private DistributedQueryRunner(
         this.trinoClient = closer.register(testingTrinoClientFactory.create(coordinator, defaultSession));
 
         ensureNodesGloballyVisible();
-        log.debug("Created DistributedQueryRunner in %s (unclosed instances = %s)", nanosSince(start), unclosedInstances.incrementAndGet());
+        log.info("Created DistributedQueryRunner in %s (unclosed instances = %s)", nanosSince(start), unclosedInstances.incrementAndGet());
     }
 
     private TestingTrinoServer createServer(

Original file line number	Diff line number	Diff line change
`@@ -218,7 +218,7 @@ private DistributedQueryRunner(`
`218`	`218`	`this.trinoClient = closer.register(testingTrinoClientFactory.create(coordinator, defaultSession));`
`219`	`219`
`220`	`220`	`ensureNodesGloballyVisible();`
`221`		`- log.debug("Created DistributedQueryRunner in %s (unclosed instances = %s)", nanosSince(start), unclosedInstances.incrementAndGet());`
	`221`	`+ log.info("Created DistributedQueryRunner in %s (unclosed instances = %s)", nanosSince(start), unclosedInstances.incrementAndGet());`
`222`	`222`	`}`
`223`	`223`
`224`	`224`	`private TestingTrinoServer createServer(`