Ticket created [1], I'll make the changes, thanks for the input everyone. On Wed, May 12, 2021 at 7:14 PM Dmitry Pavlov <dpav...@apache.org> wrote:
> +1 > > Without awareneess, on the contrary, we can get huge memory utilization on > one server node. > With awareness we're more scalable on that matter. > > Sincerely, > Dmitriy Pavlov > > On 2021/05/12 15:39:53, Igor Sapego <isap...@apache.org> wrote: > > +1 from me. There were no major issues with this feature and it gives > > good performance boost for many cases. > > > > Best Regards, > > Igor > > > > > > On Wed, May 12, 2021 at 5:18 PM Ivan Daschinsky <ivanda...@gmail.com> > wrote: > > > > > Huge +1 from me. PA should be enabled by default. > > > > > > ср, 12 мая 2021 г. в 13:33, Pavel Tupitsyn <ptupit...@apache.org>: > > > > > > > > Igniters, > > > > > > > > Partition Awareness (PA) is implemented in 5 out of 6 thin clients > [1]. > > > > > > > > However, this feature is disabled by default in most clients for > > > > compatibility reasons: > > > > initially we only used one connection to the cluster, but with PA > enabled > > > > we establish > > > > connections to every server node, which may be not desirable in some > use > > > > cases. > > > > > > > > I expect those scenarios to be rare, and in most cases it makes > sense to > > > > enable PA by default. > > > > Thoughts, objections? > > > > > > > > > > > > [1] > > > > https://cwiki.apache.org/confluence/display/IGNITE/Thin+clients+features > > > > > > > > > > > > -- > > > Sincerely yours, Ivan Daschinskiy > > > > > >
