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

Vladimir Ozerov commented on IGNITE-8915:
-----------------------------------------

[~daradurvs], [~NIzhikov], fix looks incomplete to me:
1) There is no check for {{SqlFieldsQuery}}
2) There are no tests for {{SqlFieldsQuery}} and non-REPLICATED caches
3) Why {{ScanQuery}} and {{TextQuery}} doesn't use the same check?
4) JDBC and ODBC drivers do not use {{IgniteCacheProxyImpl}}, so implemented 
check is skipped altogether. For SQL you should use 
{{GridQueryProcessor.querySqlFields}} and {{GridQueryProcessor.querySql}} 
methods. Also we need tests for JDBC.

> NPE during executing local SqlQuery from client node
> ----------------------------------------------------
>
>                 Key: IGNITE-8915
>                 URL: https://issues.apache.org/jira/browse/IGNITE-8915
>             Project: Ignite
>          Issue Type: Bug
>    Affects Versions: 2.5
>            Reporter: Vyacheslav Daradur
>            Assignee: Nikolay Izhikov
>            Priority: Major
>             Fix For: 2.7
>
>         Attachments: IgniteCacheReplicatedClientLocalQuerySelfTest.java
>
>
> NPE when trying to execute {{SqlQuery}} with {{setLocal(true)}} from client 
> node.
> [Reproducer|^IgniteCacheReplicatedClientLocalQuerySelfTest.java].
> UPD:
> Right behavior:
> Local query should be forbidden and a sensible exception should be thrown if 
> it is executed on client node.



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

Reply via email to