Re: JVM_OPTS in control.sh and ignite.sh

2020-10-02 Thread Ilya Kasnacheev
Hello! I think this is sensible approach, but please make sure that our experimental features enabling patterns are unaffected. Regards, -- Ilya Kasnacheev чт, 1 окт. 2020 г. в 14:45, Данилов Семён : > Hello! > > I added message > "JVM_OPTS environment variable is set, but will not be used. T

Re: JVM_OPTS in control.sh and ignite.sh

2020-10-01 Thread Данилов Семён
Hello! I added message "JVM_OPTS environment variable is set, but will not be used. To pass JVM options use CONTROL_JVM_OPTS" "JVM_OPTS=${JVM_OPTS}" in both control.sh and control.bat files. Cheers, Sam. 28.09.2020, 16:56, "Ilya Kasnacheev" : > Hello! > > Can we at least print a warning when co

Re: JVM_OPTS in control.sh and ignite.sh

2020-09-28 Thread Ilya Kasnacheev
Hello! Can we at least print a warning when control.sh is run and JVM_OPTS is set? Then you can use other env var. Or get rid of it and rely on -J. Regards, -- Ilya Kasnacheev пн, 28 сент. 2020 г. в 12:39, Данилов Семён : > Hello, everyone! > > Let's wrap this up somehow. > > I think that cre

Re: JVM_OPTS in control.sh and ignite.sh

2020-09-28 Thread Данилов Семён
Hello, everyone! Let's wrap this up somehow. I think that creating a different control.sh script for docker will create a precedent and we might end up with a plethora of scripts for different situations. Control.sh should be independent from ignite.sh thus using different set of environment

Re: JVM_OPTS in control.sh and ignite.sh

2020-09-24 Thread Valentin Kulichenko
Why does the control.sh use JVM_OPTS in the first place? Is there a case when a user might need to modify them? I can't think of one. -Val On Thu, Sep 24, 2020 at 6:42 AM Evgenii Zhuravlev wrote: > Ilya, > > You can get absolutely the same behaviour when you set JVM_OPTS even > without Docker.

Re: JVM_OPTS in control.sh and ignite.sh

2020-09-24 Thread Evgenii Zhuravlev
Ilya, You can get absolutely the same behaviour when you set JVM_OPTS even without Docker. Evgenii чт, 24 сент. 2020 г. в 05:44, Ilya Kasnacheev : > Hello! > > If the issue is with docker only, then maybe we should get rid of JVM_OPTS > with docker entirely? E.g. pass them as parameters. > > I'

Re: JVM_OPTS in control.sh and ignite.sh

2020-09-24 Thread Ilya Kasnacheev
Hello! If the issue is with docker only, then maybe we should get rid of JVM_OPTS with docker entirely? E.g. pass them as parameters. I'm not sold on this change yet, it breaks backward compatibility for marginal benefit. Regards, -- Ilya Kasnacheev чт, 24 сент. 2020 г. в 15:35, Данилов Семён

JVM_OPTS in control.sh and ignite.sh

2020-09-24 Thread Данилов Семён
Hello, Igniters! I recently discovered, that control.sh and ignite.sh both use JVM_OPTS environment variable. This can lead to various issues (especially in docker), such as: * Control utility will have the same xms/xmx parameters. * Control utility won't launch due to JMX port being in use (as