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

Kirill Shirokov updated IGNITE-7046:
------------------------------------
    Description: 
The following test being added to 
org.apache.ignite.internal.processors.cache.distributed.replicated.IgniteCacheReplicatedQuerySelfTest:


{noformat}
    /**
     * Verifies that in the case of client query the index is not used and a 
correct exception is thrown.
     *
     * @throws Exception If failed.
     */
    public void testClientOnlyNodeIndexException() throws Exception {
        try {
            Ignite g = startGrid("client");

            IgniteCache<Integer, Integer> c = jcache(g, Integer.class, 
Integer.class);

            try {
                List<List<?>> cres = c.query(new SqlFieldsQuery("select 
count(*) from Integer")
                    .setLocal(true)).getAll();
            } 
            catch (IgniteException e) {
                throw e; // FIXME: put an exception-checking code here instead 
of throw
            }
        }
        finally {
            stopGrid("client");
        }
    }
{noformat}

...will result in NPE instead of an Ignite exception explaining the appropriate 
cause.

  was:
The following test being added to 
org.apache.ignite.internal.processors.cache.distributed.replicated.IgniteCacheReplicatedQuerySelfTest:


{noformat}
    /**
     * Method verifies that in the case of client query index is not used and a 
correct exception is thrown.
     *
     * @throws Exception If failed.
     */
    public void testClientOnlyNodeIndexException() throws Exception {
        try {
            Ignite g = startGrid("client");

            IgniteCache<Integer, Integer> c = jcache(g, Integer.class, 
Integer.class);

            try {
                List<List<?>> cres = c.query(new SqlFieldsQuery("select 
count(*) from Integer")
                    .setLocal(true)).getAll();
            } 
            catch (IgniteException e) {
                throw e; // FIXME: put an exception-checking code here instead 
of throw
            }
        }
        finally {
            stopGrid("client");
        }
    }
{noformat}

...will result in NPE instead of an Ignite exception explaining the appropriate 
cause.


> Client queries should throw a correct exception
> -----------------------------------------------
>
>                 Key: IGNITE-7046
>                 URL: https://issues.apache.org/jira/browse/IGNITE-7046
>             Project: Ignite
>          Issue Type: Bug
>          Components: sql
>    Affects Versions: 2.3
>            Reporter: Kirill Shirokov
>
> The following test being added to 
> org.apache.ignite.internal.processors.cache.distributed.replicated.IgniteCacheReplicatedQuerySelfTest:
> {noformat}
>     /**
>      * Verifies that in the case of client query the index is not used and a 
> correct exception is thrown.
>      *
>      * @throws Exception If failed.
>      */
>     public void testClientOnlyNodeIndexException() throws Exception {
>         try {
>             Ignite g = startGrid("client");
>             IgniteCache<Integer, Integer> c = jcache(g, Integer.class, 
> Integer.class);
>             try {
>                 List<List<?>> cres = c.query(new SqlFieldsQuery("select 
> count(*) from Integer")
>                     .setLocal(true)).getAll();
>             } 
>             catch (IgniteException e) {
>                 throw e; // FIXME: put an exception-checking code here 
> instead of throw
>             }
>         }
>         finally {
>             stopGrid("client");
>         }
>     }
> {noformat}
> ...will result in NPE instead of an Ignite exception explaining the 
> appropriate cause.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to