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

Update `travel-sample` sample dataset bundled in the Beta software package

    XMLWordPrintable

Details

    • Task
    • Status: Closed
    • Major
    • Resolution: Done
    • Cheshire-Cat
    • 7.0.0
    • tools
    • 1

    Description

      Tracking ticket to update the bundled travel-sample dataset in Cheshire Cat Beta. 

      While discussions related to the sample dataset are in progress for future milestones, not having an updated sample dataset in the public Beta build will lead to a poor user experience. Hence request updating the `travel-sample` dataset (for Beta) to add named collections and a named scope. This is to ensure that all documentation examples (old and new) work OOTB. 

      Here's the updated diagram for reference: https://docs.google.com/presentation/d/1TjKhrvRnB70upU9rGWaOhwJo43EYtTuAzMqiTef6d7U/edit#slide=id.g9f5bfa070b_0_39

      Binh shared the commands to update the dataset for Phase 1 (Nov Beta): https://docs.google.com/document/d/1AP6UdN2mYoBeK4VXuBe_hJYTixr0GGEEDjL34ssL1zA/edit?ts=5fa2f0df

      Attachments

        Issue Links

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

          Activity

            Build couchbase-server-7.0.0-3733 contains docloader commit 586c1ff with commit message:
            MB-42726 travel-sample is now collection aware

            build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-3733 contains docloader commit 586c1ff with commit message: MB-42726 travel-sample is now collection aware
            michael.blow Michael Blow added a comment -

            Just a heads up: I seem to be receiving exactly 2X the mutations through DCP when consuming the default collection on travel-sample after this change- will investigate.

            expected <   "travel-sample:all:incoming_records_count" : 31591,
            actual   >   "travel-sample:all:incoming_records_count" : 63182,
            

            michael.blow Michael Blow added a comment - Just a heads up: I seem to be receiving exactly 2X the mutations through DCP when consuming the default collection on travel-sample after this change- will investigate. expected < "travel-sample:all:incoming_records_count" : 31591, actual > "travel-sample:all:incoming_records_count" : 63182,
            owend Daniel Owen added a comment - - edited

            Hi Michael Blow, This is expected as we have 2 variants of the original documents
            If you expand goproj/src/github.com/couchbase/docloader/examples/travel-sample.zip
            we have
            _default._default.route_xxxxx.json of the docs and inventory.route.route_xxxxx.json

            and similar for landmark etc.

            owend Daniel Owen added a comment - - edited Hi Michael Blow , This is expected as we have 2 variants of the original documents If you expand goproj/src/github.com/couchbase/docloader/examples/travel-sample.zip we have _default._default.route_xxxxx.json of the docs and inventory.route.route_xxxxx.json and similar for landmark etc.
            michael.blow Michael Blow added a comment -

            Hmmm I think this is not expected on the consumer side since my stream only filters on collection ID 0x0, which I confirmed in the memcached logs.

            michael.blow Michael Blow added a comment - Hmmm I think this is not expected on the consumer side since my stream only filters on collection ID 0x0, which I confirmed in the memcached logs.
            michael.blow Michael Blow added a comment - - edited

            Do we need to update the command used to load the travel-sample after this change? If I use the (documented) command that has always worked in the past, all of the documents land in the _default._default collection, whereas if I import via the web UI, the data is distributed into the various collections as expected. This could mess up any demos which use the CLI to install the travel-sample, if we are intending to break backward-compatibility with this change.

            /opt/couchbase/bin/cbdocloader -c localhost:8091 \
            -u Administrator -p password -b testBucket -m 256 \
            -d /opt/couchbase/samples/travel-sample.zip
            

            filed MB-42758 to track this issue

            michael.blow Michael Blow added a comment - - edited Do we need to update the command used to load the travel-sample after this change? If I use the ( documented ) command that has always worked in the past, all of the documents land in the _default._default collection, whereas if I import via the web UI, the data is distributed into the various collections as expected. This could mess up any demos which use the CLI to install the travel-sample, if we are intending to break backward-compatibility with this change. /opt/couchbase/bin/cbdocloader -c localhost:8091 \ -u Administrator -p password -b testBucket -m 256 \ -d /opt/couchbase/samples/travel-sample.zip filed MB-42758 to track this issue

            People

              amarantha.kulkarni Amarantha Kulkarni (Inactive)
              amarantha.kulkarni Amarantha Kulkarni (Inactive)
              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