[
https://issues.apache.org/jira/browse/IGNITE-23888?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17911103#comment-17911103
]
Pavel Tupitsyn commented on IGNITE-23888:
-----------------------------------------
[~isapego] I'm not sure if we need tests for argument validation. The message
has changed, I started fixing the test, then decided it is not worth keeping.
If you think that we need argument check tests for all the properties - let me
know, we can create a separate ticket for that.
Thanks!
Merged to main:
[d224f6fb8fdd727bd5e1dfcddceacf36d6f64ff3|https://github.com/apache/ignite-3/commit/d224f6fb8fdd727bd5e1dfcddceacf36d6f64ff3]
> Thin 3.0: Review special values in IgniteClient config
> ------------------------------------------------------
>
> Key: IGNITE-23888
> URL: https://issues.apache.org/jira/browse/IGNITE-23888
> Project: Ignite
> Issue Type: Improvement
> Components: thin client
> Reporter: Pavel Tupitsyn
> Assignee: Pavel Tupitsyn
> Priority: Major
> Labels: ignite-3, important
> Fix For: 3.0
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> Some properties accept 0 and -1 (timeouts, retries).
> Review all of them and make sure that the behavior is consistent and
> well-documented.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
