Mailing-List: contact dev-help@ignite.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of vozerov@gridgain.com
 designates 209.85.192.54 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAGcMBHjfAaT8OFiHxN5p=pmAG3R0pEXT4i3Gq-OJtbeyEuyWbw@mail.gmail.com>
References: 
 <CAGt6o8kZ2YTjjwBJrfiAk9XaFxGdbycE0NetbSpL+=NsgFmpjg@mail.gmail.com>
	<CAGcMBHjfAaT8OFiHxN5p=pmAG3R0pEXT4i3Gq-OJtbeyEuyWbw@mail.gmail.com>
Date: Wed, 4 Feb 2015 15:52:48 +0300
Message-ID: 
 <CAGt6o8mjsLc8wpfkqHCSF9fB6nx-iWHjJS0zAVUX-3nDys1M4w@mail.gmail.com>
Subject: Re: ClientConnectionConfiguration properties refactoring.
From: Vladimir Ozerov <vozerov@gridgain.com>
To: dev@ignite.incubator.apache.org
Content-Type: multipart/alternative; boundary=001a11339d1e282d20050e42aa90

--001a11339d1e282d20050e42aa90
Content-Type: text/plain; charset=UTF-8

One more point here.

Currently we have flag *IgniteConfiguration.restEnabled* which is
deprecated and it is advised to use
*IgniteConfiguration.clientConnectionConfiguration* instead.
I want to remove this flag. But the problem is that this flag is set to
"true" by default. Therefore, if I remove it, I will have to set default
value for "clientConnectionConfiguration" to "new
ClientConnectionConfiguration()" to keep REST enabled by default as
earlier. And if you want to disable REST, he will have to
"IgniteConfiguration.setClientConnectionConfigration(null)", what looks
counterintuitive.

So I think it is better to keep this flag in configuration and remove
deprecatino from it.

Any thoughts?

On Wed, Feb 4, 2015 at 12:40 PM, Yakov Zhdanov <yzhdanov@gridgain.com>
wrote:

> Agree. Just pay attention to abbreviations - message -> msg, count -> cnt,
> etc.
>
> --
> Yakov Zhdanov, Director R&D
> *GridGain Systems*
> www.gridgain.com
>
> 2015-02-04 12:27 GMT+03:00 Vladimir Ozerov <vozerov@gridgain.com>:
>
> > Folks,
> >
> > Currently "o.a.i.c.ClientConnectionConfiguration" class has lots of
> > properties and we have a plan to review/refactor them.
> >
> > Here is my proposal on that:
> > 1) Almost all properties have "rest" and "tcp" in their names. Let's
> remove
> > these prefixes altogether. E.g.: "restTcpSelectorCount" ->
> "selectorCount".
> > 2) Rename "clientMessageInterceptor" to "messageInterceptor"
> > 3) Remove "restAccessibleFolders" and corresponding client "LOG" command.
> > 4) Remove all mirrored (and already deprecated) properties from
> > IgniteConfiguration as backward compatibility is not a concern for
> initial
> > release.
> >
> > It appears that none other properties can be removed from this class
> > because Memcached client use binary protocol and may potentially need any
> > of them.
> >
> > Any comments?
> >
> > Vladimir.
> >
>

--001a11339d1e282d20050e42aa90--