You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Pavel Tupitsyn <pt...@apache.org> on 2021/05/12 10:32:53 UTC

Thin Clients: enable partition awareness by default

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

Re: Thin Clients: enable partition awareness by default

Posted by Pavel Tupitsyn <pt...@apache.org>.
Oops, here it is:

https://issues.apache.org/jira/browse/IGNITE-14719

On Fri, May 14, 2021 at 2:35 PM Ivan Daschinsky <iv...@gmail.com> wrote:

> Pavel, sorry, but where is the link to ticket?
>
> пт, 14 мая 2021 г. в 14:13, Pavel Tupitsyn <pt...@apache.org>:
> >
> > Ticket created [1], I'll make the changes, thanks for the input everyone.
> >
> > On Wed, May 12, 2021 at 7:14 PM Dmitry Pavlov <dp...@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 <is...@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 <ivandasch@gmail.com
> >
> > > wrote:
> > > >
> > > > > Huge +1 from me. PA should be enabled by default.
> > > > >
> > > > > ср, 12 мая 2021 г. в 13:33, Pavel Tupitsyn <pt...@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
> > > > >
> > > >
> > >
>
>
>
> --
> Sincerely yours, Ivan Daschinskiy
>

Re: Thin Clients: enable partition awareness by default

Posted by Ivan Daschinsky <iv...@gmail.com>.
Pavel, sorry, but where is the link to ticket?

пт, 14 мая 2021 г. в 14:13, Pavel Tupitsyn <pt...@apache.org>:
>
> Ticket created [1], I'll make the changes, thanks for the input everyone.
>
> On Wed, May 12, 2021 at 7:14 PM Dmitry Pavlov <dp...@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 <is...@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 <iv...@gmail.com>
> > wrote:
> > >
> > > > Huge +1 from me. PA should be enabled by default.
> > > >
> > > > ср, 12 мая 2021 г. в 13:33, Pavel Tupitsyn <pt...@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
> > > >
> > >
> >



-- 
Sincerely yours, Ivan Daschinskiy

Re: Thin Clients: enable partition awareness by default

Posted by Pavel Tupitsyn <pt...@apache.org>.
Ticket created [1], I'll make the changes, thanks for the input everyone.

On Wed, May 12, 2021 at 7:14 PM Dmitry Pavlov <dp...@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 <is...@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 <iv...@gmail.com>
> wrote:
> >
> > > Huge +1 from me. PA should be enabled by default.
> > >
> > > ср, 12 мая 2021 г. в 13:33, Pavel Tupitsyn <pt...@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
> > >
> >
>

Re: Thin Clients: enable partition awareness by default

Posted by Dmitry Pavlov <dp...@apache.org>.
+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 <is...@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 <iv...@gmail.com> wrote:
> 
> > Huge +1 from me. PA should be enabled by default.
> >
> > ср, 12 мая 2021 г. в 13:33, Pavel Tupitsyn <pt...@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
> >
> 

Re: Thin Clients: enable partition awareness by default

Posted by Igor Sapego <is...@apache.org>.
+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 <iv...@gmail.com> wrote:

> Huge +1 from me. PA should be enabled by default.
>
> ср, 12 мая 2021 г. в 13:33, Pavel Tupitsyn <pt...@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
>

Re: Thin Clients: enable partition awareness by default

Posted by Ivan Daschinsky <iv...@gmail.com>.
Huge +1 from me. PA should be enabled by default.

ср, 12 мая 2021 г. в 13:33, Pavel Tupitsyn <pt...@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