Sync JVM flags with Trino defaults

[nineinchnick]

No description provided.

[nineinchnick]

As expected, older versions fail to run with:

Unrecognized VM option 'G1NumCollectionsKeepPinned=10000000'

I could add a condition on the image.tag, but when using custom-built images, it doesn't have to be a version number. It's not possible to specify a different app version when installing the chart, so we can't use this field.

Maybe we should consider removing the default JVM flags from the chart and rely on those in the image, unless explicitly set?

[mosabua]

I agree with the idea of removing the default config and relying on the docker container defaults. Users can still override it and we then have to just ensure its documented that you take the default from the container you use and update as desired.

[nineinchnick]

I've tested this on the Helm playground to make sure it works with arbitrary values in all the referenced fields.

Stale

[nineinchnick]

I tried preparing another PR that would not override the default jvm.config but it's tricky, because we include values from the .Values.coordinator.jvm section. Handling this would require complex logic and IMO it's not worth it.