[jira] [Commented] (IGNITE-4324) ScanQuery throws incomprehensible exception when topology does not contain data nodes

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

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

Nikolay Tikhonov commented on IGNITE-4324:
------------------------------------------

[~williamdo],
Thank you for your contribution! I've reviewed your changes and have only minor notes. I've fixed them (code style and added test for partition cache). I've pushed changes to ignite-4324 branch. Could you look at it? If you don't have any objections, I'll merge the branch to master.

> ScanQuery throws incomprehensible exception when topology does not contain data nodes
> -------------------------------------------------------------------------------------
>
>                 Key: IGNITE-4324
>                 URL: https://issues.apache.org/jira/browse/IGNITE-4324
>             Project: Ignite
>          Issue Type: Bug
>          Components: cache
>    Affects Versions: 1.7, 1.8
>            Reporter: Nikolay Tikhonov
>            Assignee: William Do
>              Labels: newbie
>             Fix For: 2.1
>
>         Attachments: Tests.patch
>
>
> ScanQuery throws incomprehensible exception when topology does not contain data nodes (for example with node filter).  See attached test.
> {code}
> java.lang.IllegalArgumentException: bound must be positive
> 	at java.util.Random.nextInt(Random.java:388)
> 	at org.apache.ignite.internal.util.lang.GridFunc.rand(GridFunc.java:677)
> 	at org.apache.ignite.internal.processors.cache.query.GridCacheQueryAdapter.nodes(GridCacheQueryAdapter.java:582)
> 	at org.apache.ignite.internal.processors.cache.query.GridCacheQueryAdapter.executeScanQuery(GridCacheQueryAdapter.java:527)
> 	at org.apache.ignite.internal.processors.cache.GridCacheAdapter.igniteIterator(GridCacheAdapter.java:4119)
> 	at org.apache.ignite.internal.processors.cache.GridCacheAdapter.igniteIterator(GridCacheAdapter.java:4094)
> 	at org.apache.ignite.internal.processors.cache.IgniteCacheProxy.iterator(IgniteCacheProxy.java:1979)
> 	at org.apache.ignite.internal.CacheFilterQueryTest.testScanQuery(CacheFilterQueryTest.java:90)
> 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> 	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> 	at java.lang.reflect.Method.invoke(Method.java:498)
> 	at junit.framework.TestCase.runTest(TestCase.java:176)
> 	at org.apache.ignite.testframework.junits.GridAbstractTest.runTestInternal(GridAbstractTest.java:1768)
> 	at org.apache.ignite.testframework.junits.GridAbstractTest.access$000(GridAbstractTest.java:118)
> 	at org.apache.ignite.testframework.junits.GridAbstractTest$4.run(GridAbstractTest.java:1706)
> 	at java.lang.Thread.run(Thread.java:745)
> [
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)