cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Edward Capriolo (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-12627) Provide new seed providers
Date Sat, 05 Nov 2016 00:35:58 GMT

    [ https://issues.apache.org/jira/browse/CASSANDRA-12627?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15638193#comment-15638193
] 

Edward Capriolo commented on CASSANDRA-12627:
---------------------------------------------

I am a bit confused. I am looking at the code you are sending me here:
{quote}
 String priamSeeds = DataFetcher.fetchData("http://127.0.0.1:8080/Priam/REST/v1/cassconfig/get_seeds");
            for (String seed : priamSeeds.split(","))
                seeds.add(InetAddress.getByName(seed));
{quote}

With the feature I am proposing you can replace that with:

CASSANDRA_SEED_LIST=`wget http://127.0.0.1:8080/Priam/REST/v1/cassconfig/get_seeds` bin/cassandra

You could also do

CASSANDRA_SEED_LIST=`dig -t txt mydomain.com` bin/cassandra

Doesn't this seem extremely useful and straight forward?

> Provide new seed providers
> --------------------------
>
>                 Key: CASSANDRA-12627
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-12627
>             Project: Cassandra
>          Issue Type: New Feature
>            Reporter: Edward Capriolo
>            Assignee: Edward Capriolo
>
> SeedProvider is plugable, however only one implementation exists.
> Changes:
> * Create a SeedProvider that reads properties from System properties or env
> * Provide a SeedProvider that scans ranges of IP addresses to find peers.
> * Refactor interface to abstract class because all seed providers must provide a constructor
that accepts Map<String,String> 
> * correct error messages
> * Do not catch Exception use MultiCatch and catch typed exceptions



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message