[GitHub] couchdb-couch-index pull request: Continue to reopen DB and enum c...

[davisp <gi...@git.apache.org>]

Github user davisp commented on the pull request:

    https://github.com/apache/couchdb-couch-index/pull/14#issuecomment-216579991
  
    This turned out to be an awfully big change. In hindsight I don't think we're going to get much benefit out of reopening the database (I made that assumption almost four years ago so forgive me). Though what might be useful is a check in the handle_info clause when we get the updated index state. Something like this maybe: https://gist.github.com/davisp/153c239cfcd8a4dc53be2e43e84d280d
    
    @janl I'm not sure what you mean by fundamentally changes how views work. This change just makes it so that we restart the indexer rather than wait for the message that a client is still waiting. The technical difference would be that if you had a single view read during db updates, it would update until all active updates were handled instead of handling an arbitrary number of them and waiting for another view read.
    
    Views have always been able to stall the compaction swap. I guess this would lessen the read load to hit that condition though.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---