[
https://issues.apache.org/jira/browse/IGNITE-7285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16957198#comment-16957198
]
Pavel Tupitsyn edited comment on IGNITE-7285 at 10/22/19 4:08 PM:
------------------------------------------------------------------
[~samaitra]
* Please see comment on GitHub
* Please update {{SqlFieldsQuery}} and {{QueryOptions}} classes in .NET
accordingly - set {{Timeout}} to {{TimeSpan.FromMilliseconds(-1)}} by default
* Full test coverage for those additions is necessary on the .NET part
* Ignite C++ should be updated as well. I suppose that we should update
{{query_sql.h}} and {{query_sql_fields.h}}, and pass -1 there as the timeout.
[~isapego] please confirm.
was (Author: ptupitsyn):
[~samaitra]
* Please see comment on GitHub
* Please update {{SqlFieldsQuery}} and {{QueryOptions}} classes in .NET
accordingly - set {{Timeout}} to {{TimeSpan.FromMilliseconds(-1)}} by default
* Ignite C++ should be updated as well. I suppose that we should update
{{query_sql.h}} and {{query_sql_fields.h}}, and pass -1 there as the timeout.
[~isapego] please confirm.
> Add default query timeout
> -------------------------
>
> Key: IGNITE-7285
> URL: https://issues.apache.org/jira/browse/IGNITE-7285
> Project: Ignite
> Issue Type: Improvement
> Components: cache, sql
> Affects Versions: 2.3
> Reporter: Valentin Kulichenko
> Assignee: Saikat Maitra
> Priority: Major
> Labels: sql-stability
> Fix For: 2.8
>
> Time Spent: 8h 20m
> Remaining Estimate: 0h
>
> Currently it's possible to provide timeout only on query level. It would be
> very useful to have default timeout value provided on cache startup. Let's
> add {{CacheConfiguration#defaultQueryTimeout}} configuration property.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
