You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/09/17 17:48:56 UTC

[GitHub] [accumulo] dlmarion commented on a change in pull request #2275: Added additional tests in CompactionIT

dlmarion commented on a change in pull request #2275:
URL: https://github.com/apache/accumulo/pull/2275#discussion_r711244505



##########
File path: core/src/main/java/org/apache/accumulo/core/iterators/SleepyIterator.java
##########
@@ -0,0 +1,80 @@
+/*

Review comment:
       I will look at moving this to a test package. For clarification, this iterator is only used in tests that are trying to make a compaction take long enough in MAC that the test thread can cancel or delete it. The iterator is not used in the main test thread, so I don't think it should make the test flaky. I'll have to look at SlowIterator to see how/where it's used and if SleepyIterator can be used in it's place or vice versa.




-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org