gianm commented on code in PR #12521:
URL: https://github.com/apache/druid/pull/12521#discussion_r874088707


##########
pom.xml:
##########
@@ -1733,7 +1733,7 @@
                             <!-- set heap size to work around 
https://github.com/travis-ci/travis-ci/issues/3396 -->
                             <argLine>
                                 ${jdk.surefire.argLine}
-                                -Xmx768m -Duser.language=en -Duser.country=US 
-Dfile.encoding=UTF-8
+                                -Xmx1280m -Duser.language=en -Duser.country=US 
-Dfile.encoding=UTF-8

Review Comment:
   I was able to reproduce OOMEs on processing tests locally. I think that the 
processing and merge buffers for groupBy tests need to stay off-heap, because 
there's a bunch of them and they're 10MB each. It adds up. I'll try organizing 
things so that they stay offheap but get freed properly.



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to