Skip to content

Commit

Permalink
Refactor unit test memory limit so it is unset by default
Browse files Browse the repository at this point in the history
A fixed limit didn't work for all use cases. This approach restores the
previous default and allows developers to configure their own if
required.
  • Loading branch information
markt-asf committed Feb 5, 2025
1 parent fdd9424 commit e56500b
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions build.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2031,6 +2031,8 @@
<property name="opens.sunrmi" value="--add-opens=java.rmi/sun.rmi.transport=ALL-UNNAMED"/>
<property name="opens.javautil" value="--add-opens=java.base/java.util=ALL-UNNAMED"/>
<property name="opens.javautilconcurrent" value="--add-opens=java.base/java.util.concurrent=ALL-UNNAMED"/>
<!-- Set memory properties (unset by default) -->
<property name="test.xmx" value="-Dnoop"/>

<macrodef name="runtests"
description="Runs the unit tests using the specified connector.
Expand All @@ -2050,11 +2052,11 @@
errorproperty="test.result.error"
failureproperty="test.result.failure"
haltonfailure="${test.haltonfailure}"
threads="${test.threads}"
maxmemory="256m">
threads="${test.threads}">

<jvmarg value="${test.jvmarg.egd}"/>
<jvmarg value="-Dfile.encoding=UTF-8"/>
<jvmarg value="${test.xmx}"/>
<jvmarg value="${runtests.librarypath}"/>
<jvmarg value="${test.formatter}"/>
<jvmarg value="-Djava.net.preferIPv4Stack=${java.net.preferIPv4Stack}"/>
Expand Down

0 comments on commit e56500b

Please sign in to comment.