dsmiley commented on code in PR #2548:
URL: https://github.com/apache/solr/pull/2548#discussion_r1664889355
##########
solr/core/src/test/org/apache/solr/update/processor/DistributedUpdateProcessorTest.java:
##########
@@ -220,4 +185,45 @@ protected boolean tryLock(int lockTimeoutMs) {
return succeeded;
}
}
+
+ private static DistributedUpdateProcessor newDurp(SolrQueryRequest req, long
lockTimeoutMs) {
+ if (lockTimeoutMs <= 0) {
+ // default
+ return new DistributedUpdateProcessor(req, null, null);
+ }
+ // customize UpdateLocks with the provided timeout. And simulate docs
taking longer to index
+ final var sleepMs = 5000;
Review Comment:
Yeah I was thinking that (this would be an improvement over the test as it
ws written). I'll do that; maybe still giving it more time in case the test is
running slow.
--
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
