Skip to content

Commit

Permalink
Amend ordering of SQL Server tests execution
Browse files Browse the repository at this point in the history
This is supposed to workaround OOM problems when multiple test instances
are initialized. This seems to be prerequisite if we want to add more
QueryRunner-based tests.
  • Loading branch information
findepi committed Apr 8, 2022
1 parent a319b0b commit 5312fc4
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions plugin/trino-sqlserver/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,14 @@

<properties>
<air.main.basedir>${project.parent.basedir}</air.main.basedir>

<!--
Project's default for air.test.parallel is 'methods'. By design, 'classes' makes TestNG run tests from one class in a single thread.
As a side effect, it prevents TestNG from initializing multiple test instances upfront, which happens with 'methods'.
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>classes</air.test.parallel>
</properties>

<dependencies>
Expand Down

0 comments on commit 5312fc4

Please sign in to comment.