[GitHub] [ignite] samaitra commented on a change in pull request #6490: IGNITE-7285 Add default query timeout

2019-10-27 Thread GitBox
samaitra commented on a change in pull request #6490: IGNITE-7285 Add default query timeout URL: https://github.com/apache/ignite/pull/6490#discussion_r339394035 ## File path:

[GitHub] [ignite] samaitra commented on a change in pull request #6490: IGNITE-7285 Add default query timeout

2019-10-27 Thread GitBox
samaitra commented on a change in pull request #6490: IGNITE-7285 Add default query timeout URL: https://github.com/apache/ignite/pull/6490#discussion_r339394098 ## File path:

[GitHub] [ignite] samaitra commented on a change in pull request #6490: IGNITE-7285 Add default query timeout

2019-10-27 Thread GitBox
samaitra commented on a change in pull request #6490: IGNITE-7285 Add default query timeout URL: https://github.com/apache/ignite/pull/6490#discussion_r339394035 ## File path:

[GitHub] [ignite] samaitra commented on a change in pull request #6490: IGNITE-7285 Add default query timeout

2019-10-27 Thread GitBox
samaitra commented on a change in pull request #6490: IGNITE-7285 Add default query timeout URL: https://github.com/apache/ignite/pull/6490#discussion_r339395082 ## File path: modules/platforms/dotnet/Apache.Ignite.Core/IgniteConfiguration.cs ## @@ -72,6 +72,11 @@ public

[GitHub] [ignite] samaitra commented on issue #6490: IGNITE-7285 Add default query timeout

2019-10-27 Thread GitBox
samaitra commented on issue #6490: IGNITE-7285 Add default query timeout URL: https://github.com/apache/ignite/pull/6490#issuecomment-546778525 > > > Additionally to comments for code lines there are some questions regarding testing approach. Actually, I do not see how different are

[GitHub] [ignite] ptupitsyn commented on a change in pull request #6490: IGNITE-7285 Add default query timeout

2019-10-27 Thread GitBox
ptupitsyn commented on a change in pull request #6490: IGNITE-7285 Add default query timeout URL: https://github.com/apache/ignite/pull/6490#discussion_r339406229 ## File path: modules/platforms/dotnet/Apache.Ignite.Core/IgniteConfiguration.cs ## @@ -72,6 +72,11 @@ public