pnowojski commented on code in PR #19445:
URL: https://github.com/apache/flink/pull/19445#discussion_r850197802


##########
pom.xml:
##########
@@ -95,13 +95,18 @@ under the License.
                
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
                
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
                <hadoop.version>2.8.5</hadoop.version>
+               <flink.XmxITCase>2048m</flink.XmxITCase>
+               <flink.XmxUnitTest>1024m</flink.XmxUnitTest>
                <!-- Need to use a user property here because the surefire
                         forkCount is not exposed as a property. With this we 
can set
                         it on the "mvn" commandline in travis. -->
-               <flink.forkCount>1C</flink.forkCount>
+               <!-- Number of forkCounts for ITCase and UnitTest should take 
into account allocated memory
+                        to the jvm (-Xmx) and the available memory on the 
machine running the test -->
+               <flink.forkCountITCase>1C</flink.forkCountITCase>
+               <flink.forkCountUnitTest>2C</flink.forkCountUnitTest>

Review Comment:
   I'm fine with that, I also wasn't sure about the 1C/2C setting in the first 
place. Will do it in a separate commit.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to