Re: [IMPROVEMENT] Use IgniteSystemProperties.getString(key) instead of System.getenv(key)

2019-04-15 Thread Alexey Kuznetsov
Thanks for responding, I will file JIRA issue shortly. On Mon, Apr 15, 2019 at 2:24 PM Павлухин Иван wrote: > Hi, > > +1 > > I failed to imagine a case when IgniteSystemProperties.getString can > make a real harm. > > пн, 15 апр. 2019 г. в 09:35, Ilya Kasnacheev : > > > > Hello! > > > > I think

Re: [IMPROVEMENT] Use IgniteSystemProperties.getString(key) instead of System.getenv(key)

2019-04-15 Thread Павлухин Иван
Hi, +1 I failed to imagine a case when IgniteSystemProperties.getString can make a real harm. пн, 15 апр. 2019 г. в 09:35, Ilya Kasnacheev : > > Hello! > > I think it does indeed makes sense to always use > *IgniteSystemProperties.getStri**ng* > > Regards, > -- > Ilya Kasnacheev > > > чт, 11

Re: [IMPROVEMENT] Use IgniteSystemProperties.getString(key) instead of System.getenv(key)

2019-04-15 Thread Ilya Kasnacheev
Hello! I think it does indeed makes sense to always use *IgniteSystemProperties.getStri**ng* Regards, -- Ilya Kasnacheev чт, 11 апр. 2019 г. в 16:40, Alexey Kuznetsov : > Hi, Igniters. > > What do you think, if we will use " IgniteSystemProperties.getString(key)" > instead of "

[IMPROVEMENT] Use IgniteSystemProperties.getString(key) instead of System.getenv(key)

2019-04-11 Thread Alexey Kuznetsov
Hi, Igniters. What do you think, if we will use " IgniteSystemProperties.getString(key)" instead of " System.getenv(key)"? IgniteSystemProperties.getString(key) method check System,properties and if not found then check for System.getenv(key). The main purpose is to use it in tests, because it