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

Aleksandr Polovtsev updated IGNITE-27774:
-----------------------------------------
    Description: 
{{BulkUpdateProducer}} is a little bit strange: it implements 
{{UpdateProducer}} which is a base class for all Catalog commands, but, unlike 
all these commands, it modifies the provided context. This makes 
{{UpdateProducer#get}} contract not clear, though it implicitly states that 
commands should not update the context (at least that's what the CatalogManager 
expects).

I propose to either fix this behavior or we can simply remove this class, 
because it is only used in one place.

> Remove BulkUpdateProducer
> -------------------------
>
>                 Key: IGNITE-27774
>                 URL: https://issues.apache.org/jira/browse/IGNITE-27774
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Aleksandr Polovtsev
>            Assignee: Aleksandr Polovtsev
>            Priority: Major
>              Labels: ignite-3
>
> {{BulkUpdateProducer}} is a little bit strange: it implements 
> {{UpdateProducer}} which is a base class for all Catalog commands, but, 
> unlike all these commands, it modifies the provided context. This makes 
> {{UpdateProducer#get}} contract not clear, though it implicitly states that 
> commands should not update the context (at least that's what the 
> CatalogManager expects).
> I propose to either fix this behavior or we can simply remove this class, 
> because it is only used in one place.



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

Reply via email to