Re: Cluster discovery via Consul

2019-04-11 Thread Saikat Maitra
Hi Denis, Ilya, Thank you so much for sharing feedback. I can understand the difficulties on maintaining different integrations and also upgrading dependencies periodically. I am good with consul integration to be part of separate repository. I will close the ticket with the info. Regards,

Re: Cluster discovery via Consul

2019-04-09 Thread Denis Magda
I would support this idea of having integrations in separate Github repositories. Ignite is already overwhelmed with a bunch of integrations which complicates testing, release procedures, maintenance. Think that we, as the community, should revisit the current project structure and move even key

Re: Cluster discovery via Consul

2019-04-09 Thread Ilya Kasnacheev
Hello! I'm not sure that we should bundle Consul intergration with Ignite. We have a huge number of integrations and most of those are poorly maintained. I think that you should develop it separately, deploy releases to maven centra;/ They will be as easily reachable as our own artifacts this

Re: Cluster discovery via Consul

2019-04-08 Thread Saikat Maitra
Hi Ilya, I wanted to raise an PR to include TCPConsulIPFinder as part of Ignite modules and wanted to discuss if consul orbitz client can be used or shall we implement it differently. Regards, Saikat On Mon, Apr 8, 2019 at 6:07 AM Ilya Kasnacheev wrote: > Hello! > > Looks like there's a

Re: Cluster discovery via Consul

2019-04-08 Thread Ilya Kasnacheev
Hello! Looks like there's a ready solution for your problem. What do you have left unsolved? Regards, -- Ilya Kasnacheev пн, 8 апр. 2019 г. в 02:42, Saikat Maitra : > Hi, > > I am looking into this issue > https://issues.apache.org/jira/browse/IGNITE-1204 and found something > similar ignite

Cluster discovery via Consul

2019-04-07 Thread Saikat Maitra
Hi, I am looking into this issue https://issues.apache.org/jira/browse/IGNITE-1204 and found something similar ignite consul cluster discovery client here [https://github.com/andrea-zanetti/ignite-consul] This library depends on consul orbitz client [ https://github.com/rickfast/consul-client]