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

Konstantin Orlov updated IGNITE-19587:
--------------------------------------
    Epic Link: IGNITE-19502

> Sql. Remove execution-related part from IgniteTable 
> ----------------------------------------------------
>
>                 Key: IGNITE-19587
>                 URL: https://issues.apache.org/jira/browse/IGNITE-19587
>             Project: Ignite
>          Issue Type: Improvement
>          Components: sql
>            Reporter: Konstantin Orlov
>            Priority: Major
>              Labels: ignite-3
>
> Currently, {{org.apache.ignite.internal.sql.engine.schema.IgniteTable}} 
> interface provides access to the internal table to access the data. This was 
> convinient because event produced by {{TableManager}} contains both table 
> object and its descriptor.
> With upcoming {{CatalogService}} this won't be the case, because catalog 
> manages objects' descriptors only. 
> We need to rework {{LogicalRelImplementor}} in a way to acquire table object 
> from a manager rather than from IgniteTable. This makes migration to 
> {{CatalogService}} possible.
> h4. Implementation Note
>  * It would be nice to leave {{LogicalRelImplementor}} synchronous, awaiting 
> objects' futures outside
>  * Method {{org.apache.ignite.internal.sql.engine.schema.IgniteTable#table}} 
> should be removed, as well as {{UpdateableTable}} from extends list of 
> {{org.apache.ignite.internal.sql.engine.schema.IgniteTableImpl}}



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

Reply via email to