[ https://issues.apache.org/jira/browse/SOLR-5852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13943346#comment-13943346 ]
Erick Erickson commented on SOLR-5852: -------------------------------------- [~markrmil...@gmail.com] Hmmm, see my comment 20-Mar at 13:09. Do you really think that supporting all those variants is worthwhile when they can all be satisfied via the current code? Or are you just saying that we need to doc that the current connection string would handle all those case? > Add CloudSolrServer helper method to connect to a ZK ensemble > ------------------------------------------------------------- > > Key: SOLR-5852 > URL: https://issues.apache.org/jira/browse/SOLR-5852 > Project: Solr > Issue Type: Improvement > Reporter: Varun Thacker > Attachments: SOLR-5852-SH.patch, SOLR-5852-SH.patch, SOLR-5852.patch, > SOLR-5852_FK.patch, SOLR-5852_FK.patch > > > We should have a CloudSolrServer constructor which takes a list of ZK servers > to connect to. > Something Like > {noformat} > public CloudSolrServer(String... zkHost); > {noformat} > - Document the current constructor better to mention that to connect to a ZK > ensemble you can pass a comma-delimited list of ZK servers like > zk1:2181,zk2:2181,zk3:2181 > - Thirdly should getLbServer() and getZKStatereader() be public? -- This message was sent by Atlassian JIRA (v6.2#6252) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org