[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16630726#comment-16630726 ] ASF GitHub Bot commented on IGNITE-7282: Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/3467 > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.7 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16630722#comment-16630722 ] Pavel Tupitsyn commented on IGNITE-7282: Merged to master: {{45abb9c7069291d1bafa4a26edf23a36cc527716}}. Cherry-picked to ignite-2.7: {{2279e687997160525ae688b87f6d402660040359}} > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.7 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16629883#comment-16629883 ] Vladimir Ozerov commented on IGNITE-7282: - [~ptupitsyn], looks good to me. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.7 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16625227#comment-16625227 ] Pavel Tupitsyn commented on IGNITE-7282: [~vozerov] done, please have a look: * Changed config to use {{ICollection}} with support of port ranges like {{myhost.com:200..210}} * Deprecated {{Host}} and {{Port}} * Implemented round-robin with global state > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.7 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16623765#comment-16623765 ] Pavel Tupitsyn commented on IGNITE-7282: [~NIzhikov] Ok then, let's try to make it in time. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.7 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16623760#comment-16623760 ] Nikolay Izhikov commented on IGNITE-7282: - [~ptupitsyn] Current date is September 30. https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.7 > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.7 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16623755#comment-16623755 ] Pavel Tupitsyn commented on IGNITE-7282: [~NIzhikov] I plan to finish with implementation by Monday, then review will take some time. When is the code freeze? > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.7 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16623511#comment-16623511 ] Nikolay Izhikov commented on IGNITE-7282: - [~ptupitsyn] Do we have a chance to resolve this ticket until the code freeze of 2.7? > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.7 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16426901#comment-16426901 ] Vladimir Ozerov commented on IGNITE-7282: - [~ptupitsyn], agree on p.2 and p.3, thank you for clarification. As far as p.1, I would say yes, they could be deprecated. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.5 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16421816#comment-16421816 ] Pavel Tupitsyn commented on IGNITE-7282: 1) I think I agree, we should be consistent with other config APIs. > instead of having three properties Host, Port and Endpoints, we can have > only one - Endpoints {{Host}} and {{Port}} are already in 2.4, do you suggest to make them obsolete (deprecated)? 2) This property applies also when only one endpoint is defined: should we reconnect in case when connection has dropped or not. 3) This is not an internal detail since user provides us with a list of endpoints. They might want to know which of these provided endpoints is in use, I think this makes the API symmetrical. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.5 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16380328#comment-16380328 ] Pavel Tupitsyn commented on IGNITE-7282: Merged with recent master. [~vozerov] please review. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.5 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16353585#comment-16353585 ] Pavel Tupitsyn commented on IGNITE-7282: All done, [~vozerov], please review. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.5 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16352313#comment-16352313 ] Pavel Tupitsyn commented on IGNITE-7282: {{EndPoint}}, {{DnsEndPoint}}, {{IPEndPoint}} classes do not play well with XML configuration, there are no parameterless contructors. We may have to switch to our own class for that. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.5 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16350206#comment-16350206 ] Pavel Tupitsyn commented on IGNITE-7282: Mostly done, config schema update is needed. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.5 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16346547#comment-16346547 ] Pavel Tupitsyn commented on IGNITE-7282: Overall feature seems to be questionable, dev list discussion stated: http://apache-ignite-developers.2346864.n4.nabble.com/Thin-client-failover-mechanism-ODBC-JDBC-td26553.html > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.5 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16345054#comment-16345054 ] Pavel Tupitsyn commented on IGNITE-7282: We should never retry any client operations, because many of them are not idempotent and can cause side effects. So the logic is: if operation has failed due to socket connection failure, reconnect the socket (on next operation?), but fail current operation (and all pending async ops) > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.5 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16344813#comment-16344813 ] Pavel Tupitsyn commented on IGNITE-7282: Branched from IGNITE-7329. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > Fix For: 2.5 > > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16344755#comment-16344755 ] Pavel Tupitsyn commented on IGNITE-7282: On the second thought, we should provide a list of host:port pairs, so multiple hosts in a single property does not work. Let's provide {{ICollection IgniteClientConfiguration.EndPoints}} property in addition to existing {{Host:Port}} pair. > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (IGNITE-7282) .NET: Thin client: Failover
[ https://issues.apache.org/jira/browse/IGNITE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16335663#comment-16335663 ] Pavel Tupitsyn commented on IGNITE-7282: Proposed design: * No changes on server * User specifies multiple hosts in {{IgniteClientConfiguration.Host}} (with a comma) * Pick a random position in the list, connect * On failure or disconnect test next position > .NET: Thin client: Failover > --- > > Key: IGNITE-7282 > URL: https://issues.apache.org/jira/browse/IGNITE-7282 > Project: Ignite > Issue Type: Improvement > Components: platforms, thin client >Affects Versions: 2.4 >Reporter: Pavel Tupitsyn >Assignee: Pavel Tupitsyn >Priority: Major > Labels: .NET > > Currently user has to manually connect to some specific Ignite server. > Implement some kind of automatic failover where Thin Client knows about > multiple nodes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)