eiri commented on issue #649: couch_mrview_changes_since_tests gen_server 
failing with unknown_info
URL: https://github.com/apache/couchdb/issues/649#issuecomment-313376311
 
 
   No, this is different, just looks the same, because it's happening during 
getting view state and the cause is also race condition.
   
   In previous case we've been crashing on old index_server exit or be gone 
already, so exceptions been `normal` (for exit reason) or `noproc`.
   
   Here exception is `unknown_info` and it is actually couch_index_updater 
server crashing (and bringing index server with it, because [this 
clause](https://github.com/apache/couchdb/blob/master/src/couch_index/src/couch_index_updater.erl#L92)
  doesn't match. It seems that the updater spawned two updating workers due to 
restart on compaction, then kept pid of the second, but got [exit 
message](https://github.com/apache/couchdb/blob/master/src/couch_index/src/couch_index_updater.erl#L196)
 from the first one. This is interesting, I'd expect this swap to be 
sequential, so I wonder if we have a bug in 
[couch_util:shutdown_sync/1](https://github.com/apache/couchdb/blob/master/src/couch_index/src/couch_index_updater.erl#L77)
 and it is not that "sync".
   
   On the other hand, why do we have a compaction happening during a test? I 
thought that new compaction daemon supposed to be turned off for eunits?
 
----------------------------------------------------------------
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