oleewere opened a new pull request #40: AMBARI-24949. Log Search: add Solr node discovery + small cleanups URL: https://github.com/apache/ambari-logsearch/pull/40 # What changes were proposed in this pull request? - add discover flag to create LBHttpSolr clients by Solr Cloud client, after the client is created, the Solr cloud client is closed. (so will close ZK connection) use case: - external solr is used for ambari cluster (logfeeders are outside) - ambari cannot get the solr urls through command.json, then it is a bit problematic to provide all the solr urls manually, also can be problematic when the hosts will change -> use solr zookeeper address which will fetch the base urls (it has done once, so if the hosts will change, logfeeders should be restarted) Added some cleanups (missing properties + renaming) ## How was this patch tested? with docker and vagrant env locally Please review @g-boros
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
