From d26dc0b229043afa5aefca239e72f40d80446917 Mon Sep 17 00:00:00 2001
From: Prashant Wason <pwason@uber.com>
Date: Fri, 31 Jan 2020 10:50:27 -0800
Subject: [PATCH] [HUDI-587] Fixed generation of jacoco coverage reports.

surefire plugin's argLine is moved into a property. This configuration allows jacoco plugin to modify the argLine to insert it's Java Agent's configuration during pre-unit-test stage.
---
 pom.xml | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/pom.xml b/pom.xml
index afadbbf8b..8bdb4a6b4 100644
--- a/pom.xml
+++ b/pom.xml
@@ -113,6 +113,7 @@
     <spark.bundle.hive.shade.prefix></spark.bundle.hive.shade.prefix>
     <utilities.bundle.hive.scope>provided</utilities.bundle.hive.scope>
     <utilities.bundle.hive.shade.prefix></utilities.bundle.hive.shade.prefix>
+    <argLine>-Xmx1024m -XX:MaxPermSize=256m</argLine>
   </properties>
 
   <scm>
@@ -235,7 +236,6 @@
           <excludes>
             <exclude>**/IT*.java</exclude>
           </excludes>
-          <argLine>-Xmx1024m -XX:MaxPermSize=256m</argLine>
         </configuration>
       </plugin>
     </plugins>
@@ -264,11 +264,6 @@
               <configuration>
                 <!-- Sets the path to the file which contains the execution data. -->
                 <destFile>${project.build.directory}/coverage-reports/jacoco-ut.exec</destFile>
-                <!--
-                    Sets the name of the property containing the settings
-                    for JaCoCo runtime agent.
-                -->
-                <propertyName>surefireArgLine</propertyName>
               </configuration>
             </execution>
             <!--