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

Ilya Shishkov updated IGNITE-23051:
-----------------------------------
    Description: 
It seems, that full block of {{GridDhtCache}} initialization in 
{{GridCacheProcessor#createCacheContext}} is redundant: there are no further 
propagation of created cache context and DHT cache [1]. In case of redundancy 
of {{GridDhtCache}}, multiple usages of method 
{{GridNearTransactionalCache#dht()}} become redundant too.

Functionality was broken since [2, 3] (before first Ignite release).

So, we have to figure out whether this is redundant code or whether we have 
*_lost some useful functionality_*.

# 
https://github.com/apache/ignite/blob/b0467938fb02cd7276109d608bb1e4ac22796598/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheProcessor.java#L1395
# 
https://github.com/apache/ignite/commit/a187be6b076066f780ae9218ecddd6026759b7d0#diff-3b4ddcccb26573f109cb50ca46cd4e93a24b05adce4235caeafff2d737733156L980
# 
https://github.com/apache/ignite/commit/501bd5c39d7626094dad2e4068f823a4e05dcf42#diff-4afbbe3454964f7016a9f115afee310420bf815f9f54daebe95d6224b6f94834R1166

  was:
It seems, that full block of {{GridDhtCache}} initialization in 
{{GridCacheProcessor#createCacheContext}} is redundant: there are no further 
propagation of created cache context and DHT cache [1].

Functionality was broken since [2, 3] (before first Ignite release).

So, we have to figure out whether this is redundant code or whether we have 
*_lost some useful functionality_*.

# 
https://github.com/apache/ignite/blob/b0467938fb02cd7276109d608bb1e4ac22796598/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheProcessor.java#L1395
# 
https://github.com/apache/ignite/commit/a187be6b076066f780ae9218ecddd6026759b7d0#diff-3b4ddcccb26573f109cb50ca46cd4e93a24b05adce4235caeafff2d737733156L980
# 
https://github.com/apache/ignite/commit/501bd5c39d7626094dad2e4068f823a4e05dcf42#diff-4afbbe3454964f7016a9f115afee310420bf815f9f54daebe95d6224b6f94834R1166


> Investigate GridDhtCache redundancy
> -----------------------------------
>
>                 Key: IGNITE-23051
>                 URL: https://issues.apache.org/jira/browse/IGNITE-23051
>             Project: Ignite
>          Issue Type: Sub-task
>            Reporter: Ilya Shishkov
>            Priority: Minor
>              Labels: ise
>
> It seems, that full block of {{GridDhtCache}} initialization in 
> {{GridCacheProcessor#createCacheContext}} is redundant: there are no further 
> propagation of created cache context and DHT cache [1]. In case of redundancy 
> of {{GridDhtCache}}, multiple usages of method 
> {{GridNearTransactionalCache#dht()}} become redundant too.
> Functionality was broken since [2, 3] (before first Ignite release).
> So, we have to figure out whether this is redundant code or whether we have 
> *_lost some useful functionality_*.
> # 
> https://github.com/apache/ignite/blob/b0467938fb02cd7276109d608bb1e4ac22796598/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheProcessor.java#L1395
> # 
> https://github.com/apache/ignite/commit/a187be6b076066f780ae9218ecddd6026759b7d0#diff-3b4ddcccb26573f109cb50ca46cd4e93a24b05adce4235caeafff2d737733156L980
> # 
> https://github.com/apache/ignite/commit/501bd5c39d7626094dad2e4068f823a4e05dcf42#diff-4afbbe3454964f7016a9f115afee310420bf815f9f54daebe95d6224b6f94834R1166



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

Reply via email to