[jira] [Commented] (IGNITE-8044) IgniteQueryGenerator.getOptions() method should properly handle empty list of parameters.

2018-04-02 Thread Vyacheslav Koptilin (JIRA) 

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

Vyacheslav Koptilin commented on IGNITE-8044:
-

I have checked, the test and it doesn't seem that the test failure related to 
the described changes.

> IgniteQueryGenerator.getOptions() method should properly handle empty list of 
> parameters.
> -
>
> Key: IGNITE-8044
> URL: https://issues.apache.org/jira/browse/IGNITE-8044
> Project: Ignite
>  Issue Type: Bug
>  Components: general
>Affects Versions: 2.4
>Reporter: Vyacheslav Koptilin
>Assignee: Vyacheslav Koptilin
>Priority: Minor
> Fix For: 2.5
>
>
> {{IgniteQueryGenerator.getOptions()}} method does not check that the 
> parameters list can be empty.
> Initial discussion of the issue is available on the user-list: 
> http://apache-ignite-users.70518.x6.nabble.com/ArrayIndexOutOfBoundsException-1-in-IgniteQueryGenerator-getOptions-tt20801.html



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

[jira] [Commented] (IGNITE-8044) IgniteQueryGenerator.getOptions() method should properly handle empty list of parameters.

2018-03-30 Thread Denis Magda (JIRA) 

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

Denis Magda commented on IGNITE-8044:
-

[~slava.koptilin], [~dpavlov], the changes look good to me. Please feel free to 
merge them once you solve all the concerns about the flacky tests.

> IgniteQueryGenerator.getOptions() method should properly handle empty list of 
> parameters.
> -
>
> Key: IGNITE-8044
> URL: https://issues.apache.org/jira/browse/IGNITE-8044
> Project: Ignite
>  Issue Type: Bug
>  Components: general
>Affects Versions: 2.4
>Reporter: Vyacheslav Koptilin
>Assignee: Vyacheslav Koptilin
>Priority: Minor
> Fix For: 2.5
>
>
> {{IgniteQueryGenerator.getOptions()}} method does not check that the 
> parameters list can be empty.
> Initial discussion of the issue is available on the user-list: 
> http://apache-ignite-users.70518.x6.nabble.com/ArrayIndexOutOfBoundsException-1-in-IgniteQueryGenerator-getOptions-tt20801.html



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

[jira] [Commented] (IGNITE-8044) IgniteQueryGenerator.getOptions() method should properly handle empty list of parameters.

2018-03-29 Thread Dmitriy Pavlov (JIRA) 

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

Dmitriy Pavlov commented on IGNITE-8044:


[~slava.koptilin], Suspicious test:
   Ignite Platform .NET [ tests 1 TIMEOUT ] 
Test failures count is low < 3, probably new test introduced   exe: 
CacheQueriesTest.TestScanQuery(True) 



> IgniteQueryGenerator.getOptions() method should properly handle empty list of 
> parameters.
> -
>
> Key: IGNITE-8044
> URL: https://issues.apache.org/jira/browse/IGNITE-8044
> Project: Ignite
>  Issue Type: Bug
>  Components: general
>Affects Versions: 2.4
>Reporter: Vyacheslav Koptilin
>Assignee: Vyacheslav Koptilin
>Priority: Minor
> Fix For: 2.5
>
>
> {{IgniteQueryGenerator.getOptions()}} method does not check that the 
> parameters list can be empty.
> Initial discussion of the issue is available on the user-list: 
> http://apache-ignite-users.70518.x6.nabble.com/ArrayIndexOutOfBoundsException-1-in-IgniteQueryGenerator-getOptions-tt20801.html



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

[jira] [Commented] (IGNITE-8044) IgniteQueryGenerator.getOptions() method should properly handle empty list of parameters.

2018-03-29 Thread Vyacheslav Koptilin (JIRA) 

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

Vyacheslav Koptilin commented on IGNITE-8044:
-

TC looks good to me. I could not find test failures related to the change.
https://ci.ignite.apache.org/project.html?projectId=IgniteTests24Java8_IgniteTests24Java8=pull%2F3705%2Fhead

[~dpavlov] Could you please take a look?

> IgniteQueryGenerator.getOptions() method should properly handle empty list of 
> parameters.
> -
>
> Key: IGNITE-8044
> URL: https://issues.apache.org/jira/browse/IGNITE-8044
> Project: Ignite
>  Issue Type: Bug
>  Components: general
>Affects Versions: 2.4
>Reporter: Vyacheslav Koptilin
>Assignee: Vyacheslav Koptilin
>Priority: Minor
> Fix For: 2.5
>
>
> {{IgniteQueryGenerator.getOptions()}} method does not check that the 
> parameters list can be empty.
> Initial discussion of the issue is available on the user-list: 
> http://apache-ignite-users.70518.x6.nabble.com/ArrayIndexOutOfBoundsException-1-in-IgniteQueryGenerator-getOptions-tt20801.html



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