Uploaded image for project: 'Couchbase Gateway'
  1. Couchbase Gateway
  2. CBG-884

2.7.4 Backport CBG-819 - Generate empty delta as {} instead of null

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.5.0
    • Fix Version/s: 2.7.4
    • Component/s: SyncGateway
    • Security Level: Public
    • Labels:
      None
    • Sprint:
      CBG Sprint 57
    • Story Points:
      1

      Description

      A delta between two documents that have no changes is currently producing a value of null, but should produce a delta value of {}

      This is causing issues for CBL recieving a delta for an empty change (CBL-833)

       

      The fix for this is special handling specific to unchanged documents/roots, so take care when changing, because the main algorithm is recursive - unchanged objects that are nested should still be omitted from the delta. Maybe read a `null` value and transform it at a higher level for the root.

        Attachments

          Issue Links

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

            Activity

            Hide
            build-team Couchbase Build Team added a comment -

            Build sync_gateway-2.8.0-368 contains sync_gateway commit e9f12b8 with commit message:
            CBG-884: Backport of CBG-819 (#4777)

            Show
            build-team Couchbase Build Team added a comment - Build sync_gateway-2.8.0-368 contains sync_gateway commit e9f12b8 with commit message: CBG-884 : Backport of CBG-819 (#4777)

              People

              Assignee:
              jacques.rascagneres Jacques Rascagneres
              Reporter:
              ben.brooks Ben Brooks
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Gerrit Reviews

                  There are no open Gerrit changes

                    PagerDuty