[GitHub] [couchdb] jiangphcn commented on issue #2130: [WIP] close LRU by database path

[GitBox <gi...@apache.org>]

jiangphcn commented on issue #2130: [WIP] close LRU by database path
URL: https://github.com/apache/couchdb/pull/2130#issuecomment-546357883
 
 
   @theburge thanks for your valuable comments on these two PRs and your patience. I already logic to force to close writer when getting IOException https://github.com/cloudant-labs/clouseau/pull/23/commits/ff6d49619bb6f8196d40af14c1f1064d578f3e80
   
   Regarding to combine two RPC calls (`close_lru` and `rename`) into one, I think that it is good idea. But I am getting some difficulty to combine them because they are sent to different process reference (two services in clouseau side)
    - `rpc({main, clouseau()}, {close_lru_by_path, DbName}).` i.e. IndexCleanupService in clouseau
    - `gen_server:cast({cleanup, clouseau()}, {rename, DbName})`  i.e. IndexManagerService in clouseau
   
   Back to race condition scenario, I am thinking that it is possible to have `close_lru`, `open`, and `rename` index calling sequence. However, how much chance to have these calls against the same shared database. 
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services