Remove the default un-overridable -Xms128m value

[aivinog1]

Is your feature request related to a problem? Please describe.
It is hard to tune the JVM if I cannot override the default -Xms128m. I can't override it via JAVA_OPTS because it is used at first, and then goes explicit -Xms128m (and finally applied):

exec "$JAVACMD" $JAVA_OPTS -Xms128m -XX:+ExitOnOutOfMemoryError -Dfile.encoding=UTF-8...

Describe the solution you'd like
I would like to move all JVM options into the default Docker environment variable (JAVA_OPTS) so that users could override it easily (-Xms included).

Describe alternatives you've considered
We could just remove -Xms128m from the default options, but I can't predict the consequences.

Additional context
See the details here.
I would like to see the changes backported in 8.0, 8.1, 8.2 as well 🙂

[remcowesterhoud]

@megglos Do you know which team should be responsible for this one? I feel like this is more of a shared responsibility and I don't see it listed in the team split document.

[megglos]

this affects the dist module (which is shared? 😅 ), the script is generated via the appassembler-maven-plugin, which has a feature request for exactly this open since 2016 😅
mojohaus/appassembler#48

Just synced with @Zelldon on this, the best would probably be to remove the Xms flag from the plugin config and stick to jvm defaults
https://github.com/camunda/zeebe/blob/main/dist/pom.xml#L290

[remcowesterhoud]

Oh right, I don't know how I missed that in the split document 🤦

I'll assign it to both our project in that case

[aivinog1]

Hey @remcowesterhoud! If this is not occupied I could provide the PR to fix it:

Just synced with @Zelldon on this, the best would probably be to remove the Xms flag from the plugin config and stick to jvm defaults

[remcowesterhoud]

@aivinog1 Thanks! It's not occupied so go ahead. I'll assign the issue to you 👍