jnturton commented on pull request #2447: URL: https://github.com/apache/drill/pull/2447#issuecomment-1027942224
@vvysotskyi sorry, I wrote an @vdiravka but it was you who commented. The idea behind this GC is to reduce memory usage in low mem environments like our CI. The trade off is that it has some worse pause times than the newer GCs but some longer pauses are better than a crashed CI. My prediction is that reverting it will bring back "JVM process terminated unexpectedly" for the JDK 8 build specifically. But it can be set in another PR, I just needed a CI pass here for deps updates reasons. It was just a means to an end for me. -- 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: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
