dsmiley commented on code in PR #1247:
URL: https://github.com/apache/solr/pull/1247#discussion_r1057006582
##########
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:
The casing doesn't match the [Google Java
Style](https://google.github.io/styleguide/javaguide.html#s5.3-camel-case).
Use "Ttl" at the end.
Why not "with" naming?
##########
solr/solrj/src/java/org/apache/solr/client/solrj/impl/CloudHttp2SolrClient.java:
##########
@@ -147,6 +147,7 @@ public static class Builder {
private ResponseParser responseParser;
private long retryExpiryTime =
TimeUnit.NANOSECONDS.convert(3, TimeUnit.SECONDS); // 3 seconds or 3
million nanos
+ private int timeToLive = 60;
Review Comment:
The name should reflect the units in a suffix
--
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
