From f802d4400b14a4b11be6f6a1b758e1bb6fbb62a3 Mon Sep 17 00:00:00 2001
From: Raymond Xu <2701446+xushiyan@users.noreply.github.com>
Date: Wed, 20 May 2020 05:07:30 -0700
Subject: [PATCH] [MINOR] Fix resource cleanup in TestTableSchemaEvolution
 (#1640)

- Remove Xms it is not needed.
- extending process exit timeout from 30 to 120 sec should be safe to do
---
 .../org/apache/hudi/client/TestTableSchemaEvolution.java    | 6 +++---
 pom.xml                                                     | 3 ++-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/hudi-client/src/test/java/org/apache/hudi/client/TestTableSchemaEvolution.java b/hudi-client/src/test/java/org/apache/hudi/client/TestTableSchemaEvolution.java
index f9e59c9ab..d3081e8bd 100644
--- a/hudi-client/src/test/java/org/apache/hudi/client/TestTableSchemaEvolution.java
+++ b/hudi-client/src/test/java/org/apache/hudi/client/TestTableSchemaEvolution.java
@@ -75,13 +75,13 @@ public class TestTableSchemaEvolution extends TestHoodieClientBase {
       + TRIP_SCHEMA_SUFFIX;
 
   @BeforeEach
-  public void setUp() throws Exception {
+  public void setUp() throws IOException {
     initResources();
   }
 
   @AfterEach
-  public void tearDown() {
-    cleanupSparkContexts();
+  public void tearDown() throws IOException {
+    cleanupResources();
   }
 
   @Test
diff --git a/pom.xml b/pom.xml
index b2792f000..9fe27d788 100644
--- a/pom.xml
+++ b/pom.xml
@@ -245,7 +245,8 @@
         <version>${maven-surefire-plugin.version}</version>
         <configuration>
           <skip>${skipUTs}</skip>
-          <argLine>-Xms256m -Xmx2g</argLine>
+          <argLine>-Xmx2g</argLine>
+          <forkedProcessExitTimeoutInSeconds>120</forkedProcessExitTimeoutInSeconds>
           <systemPropertyVariables>
             <log4j.configuration>
               ${surefire-log4j.file}