[jira] [Comment Edited] (IGNITE-8915) NPE during executing local SqlQuery from client node

["Nikolay Izhikov (JIRA)" <ji...@apache.org>]

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

Nikolay Izhikov edited comment on IGNITE-8915 at 7/23/18 1:18 PM:
------------------------------------------------------------------

[~vozerov]

> 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.

If I understand you correctly by {{JDBC}} you mean JdbcClientDriver. 
Only it has {{local}} param.
Currently, we can use server {{nodeId}} only for this type of driver.
You can look at test {{JdbcConnectionSelfTest.testClientNodeId}}
So, no additional fix required here.

Can you confirm it?


was (Author: nizhikov):
[~vozerov]

> 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.

If I understand you correctly by {JDBC} you mean JdbcClientDriver. 
Only it has {local} param.
Currently, we can use server {nodeId} only for this type of driver.
You can look at test {JdbcConnectionSelfTest.testClientNodeId}
So, no additional fix required here.

Can you confirm it?

> 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)