Re: [DISCUSSION] IEP-95 Client Partition Awareness (Ignite 3)

2022-09-15 Thread Pavel Tupitsyn
Mirza, This is a good point. I assumed that assignment change is caused by a topology change, like in Ignite 2.x, and this affects all tables, so one flag is enough. In Ignite 3, though, assignment can change due to configuration update (number of replicas, etc), and this affects only a single

Re: [DISCUSSION] IEP-95 Client Partition Awareness (Ignite 3)

2022-09-15 Thread Mirza Aliev
Pavel, thank you for the IEP! I've read the IEP and checked PR briefly, and it is not clear to me how a client will determine which table's assignment it should update. I can assume that once a client receives the flag that indicates that assignments for some table have been changed, it will

A new feedback has been added : 90

2022-09-15 Thread Bugyard
A new feedback has been added, go to bugyard.io to see all the details... https://bugyard.io A new feedback has been added "previous version of documentation is better than this. this version has many category that confuse person "   by f.like.forghon View feedback

Re: [DISCUSSION] IEP-95 Client Partition Awareness (Ignite 3)

2022-09-15 Thread Pavel Tupitsyn
Igor, > not clear how a server would know when assignment has changed On the server, TableManager [1] maintains the current assignment. I've made some changes there (see the linked POC) to subscribe to those updates. However, as I understand, this logic is not yet complete and will be changed

Re: [DISCUSSION] IEP-95 Client Partition Awareness (Ignite 3)

2022-09-15 Thread Igor Sapego
Pavel, Great, this feature showed great results in Ignite 2, so It's a good idea to implement it in Ignite 3 as well. The IEP itself looks good to me, except it's not clear how a server would know when assignment has changed. Regardless of the Tracking Assignment Changes section, I like the

[DISCUSSION] IEP-95 Client Partition Awareness (Ignite 3)

2022-09-15 Thread Pavel Tupitsyn
Igniters, Please review the IEP for client partition awareness in Ignite 3 and let me know what you think. [1] https://cwiki.apache.org/confluence/display/IGNITE/IEP-95+Client+Partition+Awareness [2] https://github.com/apache/ignite-3/pull/1080