Thin Clients: enable partition awareness by default

[Pavel Tupitsyn]

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

[Ivan Daschinsky]

Huge +1 from me. PA should be enabled by default.

ср, 12 мая 2021 г. в 13:33, Pavel Tupitsyn <[hidden email]>:

>
> 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

[Igor Sapego-2]

+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 <[hidden email]> wrote:

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

[Dmitry Pavlov]

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

[Pavel Tupitsyn]

Ticket created [1], I'll make the changes, thanks for the input everyone.

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

[Ivan Daschinsky]

Pavel, sorry, but where is the link to ticket?

пт, 14 мая 2021 г. в 14:13, Pavel Tupitsyn <[hidden email]>:

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

[Pavel Tupitsyn]

Oops, here it is:

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

On Fri, May 14, 2021 at 2:35 PM Ivan Daschinsky <[hidden email]> wrote:

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