cpoerschke commented on code in PR #99:
URL: https://github.com/apache/lucene/pull/99#discussion_r1508783653
##########
lucene/monitor/src/java/org/apache/lucene/monitor/Monitor.java:
##########
@@ -55,7 +53,7 @@ public class Monitor implements Closeable {
private final long commitBatchSize;
- private final ScheduledExecutorService purgeExecutor;
+ private final PurgeScheduledTask purgeScheduledTask;
Review Comment:
Hello. I just tried to resolve the merge conflict in this pull request (and
am not really familiar with this area of the code) and it seems the #679
changes relocated the purging logic. Don't know how that impacts on the use
case here, just cross-referencing artifacts. Hope that helps.
--
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...@lucene.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
