jcoglan commented on PR #8495: URL: https://github.com/apache/pouchdb/pull/8495#issuecomment-4144234591
Hi folks 👋 Thank you for identifying this issue and all the work you've already done on it. We're doing some work on testing and fixing this now, starting with adding automated multi-tab tests here: [9212](https://github.com/apache/pouchdb/pull/9212) We have confirmed the issue exists and we can reproduce it reliably with two `PouchDB` instances in different pages, not in the same page -- that's why a unit test running inside a single page will not detect the problem. The test we've added in the above PR does reliably fail on `indexeddb`. The changes made here stop `put()` from throwing an error, but the doc does not seem to actually get written. The `update_seq` changes, but no actual doc is added to the database. The test we've written still fails, because even though `put()` returns `{ ok: true }`, `doc_count` is still lower than expected and instances still see stale `info()` output sometimes. Any insight you can provide into why this might be the case, or suggestions for properly fixing it, would be greatly appreciated :) -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
