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

[BP 7.0.1 MB-46742] - closeForRecovery does not release gCtx

    XMLWordPrintable

Details

    • Untriaged
    • 1
    • Unknown

    Attachments

      Issue Links

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          Wayne Siu Raju Suravarjjala Lynn Straus.  Please approve this Backport to 7.0.1. Thanks.

          srinath.duvuru Srinath Duvuru added a comment - Wayne Siu Raju Suravarjjala Lynn Straus .  Please approve this Backport to 7.0.1. Thanks.
          wayne Wayne Siu added a comment -

          Srinath Duvuru

          How do we verify the fix?  Any new tests from QE or Perf are required?  Thanks.

          cc: Ritam Sharma Ritesh Agarwal Mihir Kamdar Vikas Chaudhary

          wayne Wayne Siu added a comment - Srinath Duvuru How do we verify the fix?  Any new tests from QE or Perf are required?  Thanks. cc: Ritam Sharma Ritesh Agarwal Mihir Kamdar Vikas Chaudhary

          Wayne Siu We have a unit test to test the Fix (TestGlobalWorkContextForRecovery). This is in error path, not in the performance path.

          saptarshi.sen Saptarshi Sen added a comment - Wayne Siu  We have a unit test to test the Fix (TestGlobalWorkContextForRecovery). This is in error path, not in the performance path.
          wayne Wayne Siu added a comment -

          Saptarshi Sen

          Ok.  Thanks.  Added it to 7.0.1.

          wayne Wayne Siu added a comment - Saptarshi Sen Ok.  Thanks.  Added it to 7.0.1.

          Build couchbase-server-7.0.1-5966 contains plasma commit 02487ad with commit message:
          MB-47205 [BP] : Release gCtx's sCtx & Buffer Memory on closeForRecovery

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.1-5966 contains plasma commit 02487ad with commit message: MB-47205 [BP] : Release gCtx's sCtx & Buffer Memory on closeForRecovery

          There is a unit test to verify the fix. No need for further testing.

          srinath.duvuru Srinath Duvuru added a comment - There is a unit test to verify the fix. No need for further testing.

          People

            saptarshi.sen Saptarshi Sen
            saptarshi.sen Saptarshi Sen
            Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty