Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Currently chhtpd_db db_req(#httpd

      {method='POST',path_parts=[_,<<"_revs_diff">>]}

      .. does't handle a case where fabric:get_missing_revs returns

      {error, timeout}

      , producing an error: badmatch, reason:

      {error,timeout}

      ..<<"chttpd_db:db_req/2

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user mayya-sharipova opened a pull request:

          https://github.com/apache/couchdb-chttpd/pull/148

          handle timeout in _revs_diff

          when fabric workers timeout,
          fabric:get_missing_revs returns

          {error,timeout}
          this handles timeout errors

          COUCHDB-3220

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/cloudant/couchdb-chttpd 60532-handle-timeout_revs_diff

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/couchdb-chttpd/pull/148.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #148

          ----
          commit 72e1271da26b94d7e2967128e9b2a3605c1d28cc
          Author: Mayya Sharipova <mayyas@ca.ibm.com>
          Date: 2016-11-02T16:40:48Z

          handle timeout in _revs_diff

          when fabric workers timeout,
          fabric:get_missing_revs returns {error,timeout}

          this handles timeout errors

          COUCHDB-3220


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user mayya-sharipova opened a pull request: https://github.com/apache/couchdb-chttpd/pull/148 handle timeout in _revs_diff when fabric workers timeout, fabric:get_missing_revs returns {error,timeout} this handles timeout errors COUCHDB-3220 You can merge this pull request into a Git repository by running: $ git pull https://github.com/cloudant/couchdb-chttpd 60532-handle-timeout_revs_diff Alternatively you can review and apply these changes as the patch at: https://github.com/apache/couchdb-chttpd/pull/148.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #148 ---- commit 72e1271da26b94d7e2967128e9b2a3605c1d28cc Author: Mayya Sharipova <mayyas@ca.ibm.com> Date: 2016-11-02T16:40:48Z handle timeout in _revs_diff when fabric workers timeout, fabric:get_missing_revs returns {error,timeout} this handles timeout errors COUCHDB-3220
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 64235ef3b40be067d8304ee97e23eae0b446337c in couchdb-chttpd's branch refs/heads/master from Mayya Sharipova
          [ https://git-wip-us.apache.org/repos/asf?p=couchdb-chttpd.git;h=64235ef ]

          handle timeout in _revs_diff

          when fabric workers timeout,
          fabric:get_missing_revs returns

          {error,timeout}

          this handles timeout errors

          COUCHDB-3220

          Show
          jira-bot ASF subversion and git services added a comment - Commit 64235ef3b40be067d8304ee97e23eae0b446337c in couchdb-chttpd's branch refs/heads/master from Mayya Sharipova [ https://git-wip-us.apache.org/repos/asf?p=couchdb-chttpd.git;h=64235ef ] handle timeout in _revs_diff when fabric workers timeout, fabric:get_missing_revs returns {error,timeout} this handles timeout errors COUCHDB-3220
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/couchdb-chttpd/pull/148

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/couchdb-chttpd/pull/148

            People

            • Assignee:
              Unassigned
              Reporter:
              mayyas Mayya Sharipova
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development