Github user rnewson commented on the issue:
https://github.com/apache/couchdb-fabric/pull/70
I'll fix this but fyi @brkolla @davisp
---
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
Github user rnewson commented on the issue:
https://github.com/apache/couchdb-fabric/pull/70
contributors need to run `make check` on their changes before submitting
for PR, let's make that a new rule k.
---
If your project is set up for it, you can reply to this email and have
Github user rnewson commented on the issue:
https://github.com/apache/couchdb-fabric/pull/70
this turned out to be broken, all tests fail that touch this code as
maybe_update_others only has a clause for stale=update_after
---
If your project is set up for it, you can reply to this
Github user brkolla commented on the issue:
https://github.com/apache/couchdb-fabric/pull/70
Yes, this is my first code change in this area, so I was anticipating this
discussion. Took me some time to understand all the internals.
---
If your project is set up for it, you can reply
Github user rnewson commented on the issue:
https://github.com/apache/couchdb-fabric/pull/70
getting closer!
---
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
Github user brkolla commented on the issue:
https://github.com/apache/couchdb-fabric/pull/70
@rnewson I have updated the code to remove the case statement while opening
the DB. Please review it again and let me know if you find any other issues.
---
If your project is set up for