You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Sergey Uttsel (Jira)" <ji...@apache.org> on 2023/01/27 15:49:00 UTC

[jira] [Commented] (IGNITE-18624) Need to use data nodes from DistributionZoneManager used instead of BaselineManager#nodes.

    [ https://issues.apache.org/jira/browse/IGNITE-18624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17681387#comment-17681387 ] 

Sergey Uttsel commented on IGNITE-18624:
----------------------------------------

I make a PoC. When we replace BaselineManager#nodes with the data nodes from DistributionZoneManager we will see a lot of flaky test.
This is because there is a delay between updating the data nodes in the physical topology that the BaselineManager uses and the data nodes in the DistributionZoneManager.
We need to investigate approach to solve these issue. We can update the tests or create a general solution.

> Need to use data nodes from DistributionZoneManager used instead of BaselineManager#nodes.
> ------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-18624
>                 URL: https://issues.apache.org/jira/browse/IGNITE-18624
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Sergey Uttsel
>            Priority: Major
>              Labels: ignite-3
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> h3. Motivation
> Need to remove BaselineManager and use dataNodes from DistributionZoneManager instead of BaselineManager#nodes. For this purpose need to add local field with dataNodes for all zones in DistributionZoneManager.
> Need to update local dataNodes together with dataNodes in metastorage and on DistributionZoneManager#start.
> h3. Definition of Done
>  # BaselineManager class is removed.
>  # Data nodes from DistributionZoneManager used instead of BaselineManager#nodes.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)