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

Compact() shouldn't trigger import errors

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Unresolved
    • 2.1.0
    • 2.6.0
    • SyncGateway
    • Security Level: Public
    • None
    • CBG Sprint 11
    • 8

    Description

      It looks like when we call Compact() and write a "_purged":true doc, this will go through the import process and throw an error about having a _ leading property (because it's one we don't check for. This is pretty much benign, as we bail out at that point, but it would be nice if we didn't complain about it and bailed out a bit earlier.

      Will try and add logs demonstrating this, but because it relies on tombstones and metadata purge, it's a pain to do a clean repro without waiting at least an hour!

      Attachments

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

        Activity

          adamf Adam Fraser added a comment -

          Should be able to repro by simulating the compact 'create/delete' process without needing to wait an hour, and make the fix to avoid import noise in that case.  

          adamf Adam Fraser added a comment - Should be able to repro by simulating the compact 'create/delete' process without needing to wait an hour, and make the fix to avoid import noise in that case.  

          Build sync_gateway-2.6.0-2 contains sync_gateway commit 280120a with commit message:
          CBG-189: Import not attempted on purged document (#3951)

          build-team Couchbase Build Team added a comment - Build sync_gateway-2.6.0-2 contains sync_gateway commit 280120a with commit message: CBG-189 : Import not attempted on purged document (#3951)

          Build sync_gateway-2.6.0-2 contains sync_gateway commit 280120a with commit message:
          CBG-189: Import not attempted on purged document (#3951)

          build-team Couchbase Build Team added a comment - Build sync_gateway-2.6.0-2 contains sync_gateway commit 280120a with commit message: CBG-189 : Import not attempted on purged document (#3951)

          People

            jacques.rascagneres Jacques Rascagneres
            James Flather James Flather
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty