From 8f651eab3a30c190c43465d41285fb50e93308cd Mon Sep 17 00:00:00 2001
From: Piotr Findeisen <piotr.findeisen@gmail.com>
Date: Wed, 8 Jun 2022 11:25:18 +0200
Subject: [PATCH] Reduce trino-tests' memory footprint

On CI it was observed to improve test times, preventing build timeouts.
---
 testing/trino-tests/pom.xml | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/testing/trino-tests/pom.xml b/testing/trino-tests/pom.xml
index 2c8ff30edbaa..918694c8e487 100644
--- a/testing/trino-tests/pom.xml
+++ b/testing/trino-tests/pom.xml
@@ -14,6 +14,16 @@
 
     <properties>
         <air.main.basedir>${project.parent.basedir}</air.main.basedir>
+
+        <!--
+          Project's default for air.test.parallel is 'methods'. By design, 'instances' runs all the methods in the same instance in the same thread,
+          but two methods on two different instances will be running in different threads.
+          As a side effect, it prevents TestNG from initializing multiple test instances upfront, which happens with 'methods'.
+          This reduces tests' memory footprint, which generally allows them to complete faster.
+          A potential downside can be long tail single-threaded execution of a single long test class.
+          TODO (https://github.com/trinodb/trino/issues/11294) remove when we upgrade to surefire with https://issues.apache.org/jira/browse/SUREFIRE-1967
+          -->
+        <air.test.parallel>instances</air.test.parallel>
     </properties>
 
     <dependencies>