epugh commented on code in PR #1247:
URL: https://github.com/apache/solr/pull/1247#discussion_r1057972873
##########
solr/solrj/src/java/org/apache/solr/client/solrj/impl/CloudLegacySolrClient.java:
##########
@@ -270,6 +271,17 @@ public Builder withLBHttpSolrClient(LBHttpSolrClient
loadBalancedSolrClient) {
return this;
}
+ /**
+ * Sets the cache ttl for DocCollection Objects cached.
+ *
+ * @param seconds ttl value in seconds
+ */
+ public Builder setCollectionCacheTTL(int seconds) {
Review Comment:
I want to go through and change the namings for all the builders at once,
after i send a email to dev@ mailing list to get consensus. As far as the
the TTL, I didn't know about the Google Java Style, and so `TTl` looked werid
to me compared to `TTL`. I wish one of our checkers had actually flagged
that I wasn't following the properly Google Java Style!
--
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: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
