[GitHub] couchdb-fabric issue #70: Update all shards with stale=update_after
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 feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] couchdb-fabric issue #70: Update all shards with stale=update_after
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 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] couchdb-fabric issue #70: Update all shards with stale=update_after
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 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] couchdb-fabric issue #70: Update all shards with stale=update_after
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 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] couchdb-fabric issue #70: Update all shards with stale=update_after
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 so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] couchdb-fabric issue #70: Update all shards with stale=update_after
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 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
