dsmiley commented on PR #4320: URL: https://github.com/apache/solr/pull/4320#issuecomment-4371187921
There shouldn't be a double `://` in a URL. And I don't think we need to worry about the resiliency offered by additional hosts for a JDBC/SQL use-case... after all, if a single host/port is good enough for a DB then surely it is/can-be for Solr too. I suggest: `jdbc:solr:http://myhost:8983?collection=mycol` And although it'd be tempting to incorporate the collection into the path... I'm unsure if that raises a v1 vs v2 issue. I suppose it'd be a reasonable option to support such if slashes exist, e.g. a valid string could also be `jdbc:solr:http://myhost:8983/solr/mycol` And BTW we needn't support ZK based connection JDBC strings long-term (Solr 11) IMO. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
