Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-26648

Add exit path to doStreamClose for the case that downstream clients have exited

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Major
    • 5.5.0
    • 5.5.0
    • gomemcached
    • None
    • Triaged
    • Unknown

    Description

      In upr feed, doStreamClose() tries to write to mutation channel to downstream, e.g., xdcr. This write could hang if downstream has exited prior.

      One way to reduce the chance of hanging is to have doStreamClose listen on feed.closer. If the downstream has exited, hopefully it would close upr feed, and doStreamClose would be able to exit.

      Attachments

        Issue Links

          blocks

          Bug - A problem which impairs or prevents the functions of the product. MB-26816 [5.1.0 CLONE MB-26648]- Add exit path to doStreamClose for the case that downstream clients have exited

          • Major - Major loss of function.
          • Closed
          relates to

          Bug - A problem which impairs or prevents the functions of the product. MB-27277 [5.0 CLONE] old dcp connection is not closed when replication restarts

          • Major - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. MB-26811 [4.6.5 CLONE MB-26648]- Add exit path to doStreamClose for the case that downstream clients have exited

          • Major - Major loss of function.
          • Closed
          For Gerrit Dashboard: MB-26648
          # Subject Branch Project Status CR V
          84869,3 MB-26648 provide an exit path for doStreamClose when feed is closed master gomemcached Status: MERGED +2 +1

          Activity

            People

              yu Yu Sui (Inactive)
              yu Yu Sui (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty