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

Andrew Mashenkov commented on IGNITE-6173:
------------------------------------------

I don't like multipurpose method 
UpdatePlanBuilder.isMvccEnabledAndStartLazyCaches(parser).
Why we can't return complex result with mvcc flag and list of lazy cache names 
and then start cache to make code more readable.
List of lazy caches expects to be short mostly. 

 

 

> SQL: do not start caches on client nodes
> ----------------------------------------
>
>                 Key: IGNITE-6173
>                 URL: https://issues.apache.org/jira/browse/IGNITE-6173
>             Project: Ignite
>          Issue Type: Task
>          Components: cache, sql
>    Affects Versions: 2.1
>            Reporter: Vladimir Ozerov
>            Assignee: Yury Gerzhedovich
>            Priority: Major
>              Labels: sql-stability
>
> When cache is started, this even is distributed through custom discovery 
> message. Server nodes start the cache, client nodes do nothing until cache is 
> requested explicitly. At the same time H2 database objects are created only 
> when cache is really started. 
> For this reason query parsing could lead to {{TABLE NOT FOUND}}, {{INDEX NOT 
> FOUND}}, etc. errors. If such exception is observed, we force start of all 
> known cache on a client and then retry. See 
> {{GridCacheProcessor#createMissingQueryCaches}} method. 
> First, client node cache start leads to another custom discovery message. So 
> query performance may suffer. Second, this is not needed! We already have all 
> necessary cache info in discovery. 
> Let's try to find a way to use available discovery data and do not start 
> cache on a client for SQL query execution.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to