You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Yury Gerzhedovich (Jira)" <ji...@apache.org> on 2022/12/08 08:56:00 UTC

[jira] [Updated] (IGNITE-18211) Sql. Adjust Affinity distribution to the new colocation rules

     [ https://issues.apache.org/jira/browse/IGNITE-18211?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Yury Gerzhedovich updated IGNITE-18211:
---------------------------------------
    Description: 
Need to adjust affinity function to be prepared to upcoming concept of Distribution Zones.

Following changes are required:
 * AffinityFunction has to be updated with id (name) of the distribution zone
 * (De-)Serialization of the DistributionFunction need to be adjusted as well
 * Integration with table

Besides, it looks like the following condition is no more valid, thus should be removed:
{code:java}
        // org.apache.ignite.internal.sql.engine.trait.TraitUtils#convertDistribution

        if (toTrait.function().affinity() && toTrait.getKeys().size() > 1) {
            return null;
        }
{code}
see: org.apache.ignite.internal.sql.engine.planner.JoinColocationPlannerTest and org.apache.ignite.internal.sqle.ngine.ItSetOpTest

  was:
Need to adjust affinity function to be prepared to upcoming concept of [Distribution Zones|https://issues.apache.org/jira/browse/IGNITE-17924].

Following changes are required:
* AffinityFunction has to be updated with id (name) of the distribution zone
* (De-)Serialization of the DistributionFunction need to be adjusted as well
* Integration with table

Besides, it looks like the following condition is no more valid, thus should be removed: 

{code:java}
        // org.apache.ignite.internal.sql.engine.trait.TraitUtils#convertDistribution

        if (toTrait.function().affinity() && toTrait.getKeys().size() > 1) {
            return null;
        }
{code}

see: org.apache.ignite.internal.sql.engine.planner.JoinColocationPlannerTest



> Sql. Adjust Affinity distribution to the new colocation rules
> -------------------------------------------------------------
>
>                 Key: IGNITE-18211
>                 URL: https://issues.apache.org/jira/browse/IGNITE-18211
>             Project: Ignite
>          Issue Type: Improvement
>          Components: sql
>            Reporter: Konstantin Orlov
>            Assignee: Pavel Pereslegin
>            Priority: Major
>              Labels: ignite-3
>
> Need to adjust affinity function to be prepared to upcoming concept of Distribution Zones.
> Following changes are required:
>  * AffinityFunction has to be updated with id (name) of the distribution zone
>  * (De-)Serialization of the DistributionFunction need to be adjusted as well
>  * Integration with table
> Besides, it looks like the following condition is no more valid, thus should be removed:
> {code:java}
>         // org.apache.ignite.internal.sql.engine.trait.TraitUtils#convertDistribution
>         if (toTrait.function().affinity() && toTrait.getKeys().size() > 1) {
>             return null;
>         }
> {code}
> see: org.apache.ignite.internal.sql.engine.planner.JoinColocationPlannerTest and org.apache.ignite.internal.sqle.ngine.ItSetOpTest



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