Then I would propose to configure it in Visor, not in Ignite. I still vote for taking it out from Ignite and filling another ticket to add it to Visor.
D. On Wed, Feb 11, 2015 at 7:46 AM, Alexey Kuznetsov <[email protected]> wrote: > Commandline VisorAlertCommand use them. > > > On Wed, Feb 11, 2015 at 10:25 PM, Dmitriy Setrakyan <[email protected] > > > wrote: > > > I don't think these settings are used anywhere in the project. Are they > > used in the management console? > > > > D. > > > > On Wed, Feb 11, 2015 at 7:18 AM, Alexey Kuznetsov < > [email protected] > > > > > wrote: > > > > > How about to implement e-mail notifications as a kind of plugin / some > > > pluggable service? > > > > > > > > > On Wed, Feb 11, 2015 at 10:14 PM, Dmitriy Setrakyan < > > [email protected] > > > > > > > wrote: > > > > > > > I would remove it from Ignite altogether. > > > > > > > > On Wed, Feb 11, 2015 at 7:07 AM, Vladimir Ozerov < > [email protected] > > > > > > > wrote: > > > > > > > > > Hi, > > > > > > > > > > We have lots of properties in IgniteConfiguration dedicated to > email > > > > setup: > > > > > > > > > > setLifeCycleEmailNotification > > > > > setSmtpSsl > > > > > setSmtpStartTls > > > > > setSmtpHost > > > > > setSmtpPort > > > > > setSmtpUsername > > > > > setSmtpPassword > > > > > setAdminEmails > > > > > setSmtpFromEmail > > > > > > > > > > It seems to me that all these properties are very rarely used, > aren't > > > > they? > > > > > If so, how about moving them to separate bean to make > > > IgniteConfiguration > > > > > lighter? > > > > > > > > > > Vladimir. > > > > > > > > > > > > > > > > > > > > > -- > > > Alexey Kuznetsov > > > GridGain Systems > > > www.gridgain.com > > > > > > > > > -- > Alexey Kuznetsov > GridGain Systems > www.gridgain.com >
