eiri opened a new pull request #614: Fix compaction daemon tests
URL: https://github.com/apache/couchdb/pull/614
 
 
   ## Overview
   
   Test `couchdb_compaction_daemon_tests` periodically crashes with database 
not been idle after compaction completion. The reason been a race condition 
when index compacting process can be a bit slow (2ms is enough) to release db 
monitor after `swap_compacted` call. There are also amount of CRASH messages in 
test log related to premature unload of meck and indexers crashes on teardown, 
which makes troubleshooting harder.
   
   This patch addresses the CRASH messages by refactoring the test's setup and 
teardown and adding monitor on compacting process to resolve the race.
   
   ## Testing recommendations
   
   `make eunit apps=couch suites=couchdb_compaction_daemon_tests 
tests=compaction_daemon_test_` should successfully pass and `couch.log` in 
`src/couch/.eunit` shouldn't have CRASH messages.
   
   ## GitHub issue number
   
   Fixes #564
   
   ## Checklist
   
   - [x] Code is written and works correctly;
   - [x] Changes are covered by tests;
   - [ ] Documentation reflects the changes;
   
 
----------------------------------------------------------------
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