eiri commented on issue #564: compaction eunit test failure: database not idle 
after compaction complete
URL: https://github.com/apache/couchdb/issues/564#issuecomment-310739835
 
 
   So, this took some chasing to figure out. The function `wait_for_compaction` 
is actually waiting on compaction _monitor_, not db compaction and since we 
have `parallel_view_compact` set to `false` in compaction daemon, the whole 
thing is sequential and doesn't have a race.
   
   There are, however, a race in view compacting process (not compactor server, 
but actual compacting worker). The issue is that we reuse same data dir for all 
the tests, so there are quite a few temp database leftovers from the previous 
tests. Since compaction daemon scans all the databases on `compact by default 
rule` test, this creates quite a crowd, so on starved resources compacting 
process could be slightly slow to release db monitor even after 
`swap_compacted` called.
   
   I've opened apache/couchdb#614 to ensure "our" compacting proc exits before 
we do the db idling test.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to