Re: [PR] CASSANDRA-13855 - Add HTTP/S Seed provider [cassandra]

["Mmuzaf (via GitHub)" <gi...@apache.org>]

Mmuzaf commented on code in PR #2988:
URL: https://github.com/apache/cassandra/pull/2988#discussion_r1537631621


##########
conf/cassandra.yaml:
##########
@@ -647,6 +647,17 @@ seed_provider:
       # based on the configured name service on the system. Defaults to "false".
       #  resolve_multiple_ip_addresses_per_dns_record: "false"
 
+
+#  - class_name: org.apache.cassandra.locator.UrlSeedProvider
+#    parameter:
+        # seeds_url points to config which container one seed per line.
+        # Ex: "http://<seeds-url-address>"
+#      - seeds_url: "http://127.0.0.1"

Review Comment:
   I think it should be a comma separated list of URLs. Having one URL is a single point of failure. Users should be able to set as many points as they like.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org