aparnasuresh85 commented on code in PR #2629:
URL: https://github.com/apache/solr/pull/2629#discussion_r1709952802
##########
solr/solrj/src/java/org/apache/solr/client/solrj/impl/BaseHttpClusterStateProvider.java:
##########
@@ -55,7 +56,8 @@ public abstract class BaseHttpClusterStateProvider implements
ClusterStateProvid
volatile Map<String, Map<String, String>> aliasProperties;
long aliasesTimestamp = 0;
- private int cacheTimeout = 5; // the liveNodes and aliases cache will be
invalidated after 5 secs
+ // the liveNodes and aliases cache will be invalidated after 5 secs
+ private int cacheTimeout =
EnvUtils.getPropertyAsInteger("solr.httpcsp.cache.timeout", 5);
Review Comment:
The timeout was originally specified in seconds, and the elapsed time—used
for comparison with the timeout value for livenodes and aliases—is also
interpreted in seconds. Therefore, I set the default value to 5 seconds. Let me
know if the interpretation and the cachetimeout value needs to be changed to
milliseconds.
--
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]
