ltapilot commented on a change in pull request #170: Fixed CLI integration tests so that they run properly on Windows. URL: https://github.com/apache/incubator-daffodil/pull/170#discussion_r251567248
########## File path: build.sbt ########## @@ -139,6 +139,7 @@ lazy val commonSettings = Seq( sourceManaged := baseDirectory.value / "src_managed", resourceManaged := baseDirectory.value / "resource_managed", libraryDependencies ++= Dependencies.common, + parallelExecution in IntegrationTest := false Review comment: I spent a couple of hours messing with this today. It turns out that the platform doesn't do a good enough job of predicting memory usage. Subsequently it starts too many tests in parallel and when the tests all ask for large blocks of RAM serious memory pressure builds at the system level. In my case the system even ran out of swap space. My system has 16GB of RAM and it is clear that this should still be able to run on systems with less than that. For these reasons I'm going to leave parallel execution OFF in this pull request. I wish it were not so, but I think that's the only way we're going to get this to run consistently and without failure. Sorry. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
