gerlowskija commented on code in PR #721: URL: https://github.com/apache/solr-operator/pull/721#discussion_r1743659762
########## helm/solr/README.md: ########## @@ -1,10 +1,8 @@ Apache Solr ============= -Apache Solr is the popular, blazing-fast, open source enterprise search platform built on Apache Lucene™. - -Solr is highly reliable, scalable and fault tolerant, providing distributed indexing, replication and load-balanced querying, automated failover and recovery, centralized configuration and more. -Solr powers the search and navigation features of many of the world's largest internet sites. +Solr is the blazing fast, open source, multi-modal search platform built on Apache Lucene. Its full-text, vector, and geospatial search power many of the world's largest organizations. +Major features include: Kubernetes and docker integration; streaming and analytics; and highlighting, faceting, and spellchecking. Review Comment: In my mind at least, there's a distinction between the lists in each sentence. The first sentence (and its list) is focused narrowly on the "types" of search that Solr supports: full-text, vector, etc. The second sentence is a more general grab-bag, to list things users might care about that aren't strictly search-related. Given that distinction - I guess i could imagine putting "analytics" (or "analytics search") in the first sentence if you feel strongly about it? But I probably wouldn't move "geospatial" to the second sentence, unless we want to ditch my distinction altogether. -- 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]
