java s2i builder appears to be missing memory limit logic

[sherl0cks]

The java run scripts seem to be missing the logic for properly setting memory limits, which could cause apps to crash unexpectedly

https://github.com/fabric8io-images/java/blob/master/images/jboss/openjdk8/jdk/container-limits
https://developers.redhat.com/blog/2017/03/14/java-inside-docker/

[rebeccaSimmonds19]

Hi @sherl0cks quick question when you originally put this issue in were you referring to the fabric8 s2i only or all the spark drivers too. This first link is no longer working. thanx for clarification

[sherl0cks]

the fabric8 s2i (which I guess was moved) has the logic in it to properly set memory values. That logic should also live in the spark s2i builders, else they will see the same issues.

…

On Oct 10 2018, at 5:02 pm, Becky ***@***.***> wrote: Hi @sherl0cks (https://github.com/sherl0cks) quick question when you originally put this issue in were you referring to the fabric8 s2i only or all the spark drivers too. This first link is no longer working. thanx for clarification — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub (#159 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/ABRfYUUwy0Fz9Z2wWNh_WeTEs3m9kNCAks5ujf3fgaJpZM4REHsI).