History log of /5.5.2/kv_engine/engines/ep/src/collections/manager.cc (Results 1 - 10 of 10)
Revision (<<< Hide revision tags) (Show revision tags >>>) Date Author Comments
Revision tags: v7.0.2, v6.6.3, v7.0.1, v7.0.0, v6.6.2, v6.5.2, v6.6.1, v6.0.5, v6.6.0, v6.5.1, v6.0.4, v6.5.0, v6.0.3, v5.5.4, v5.5.5, v5.5.6, v6.0.1, v5.5.3, v6.0.0, v5.1.3, v5.5.2, v5.5.1, v5.1.2, v5.1.1
# 13069eac 16-Mar-2018 Jim Walker <jim@couchbase.com>

MB-27121: Fail get_manifest if no manifest exists

Check the current pointer is valid (else we will make an invalid
memory access). Return the no_collections_manifest error if the
man

MB-27121: Fail get_manifest if no manifest exists

Check the current pointer is valid (else we will make an invalid
memory access). Return the no_collections_manifest error if the
manifest is null

Change-Id: Iaba5cca07773b4ea780d9943b24c258675979da3
Reviewed-on: http://review.couchbase.org/91071
Reviewed-by: Dave Rigby <daver@couchbase.com>
Reviewed-by: Tim Bradgate <tim.bradgate@couchbase.com>
Tested-by: Dave Rigby <daver@couchbase.com>

show more ...


Revision tags: v5.0.1
# 901d1ab8 05-Dec-2017 Jim Walker <jim@couchbase.com>

MB-27121: [3/3] Plumb GET_MANIFEST opcode into engines

Add the opcode and required functions so that we can accept and
process a collections get manifest command.

Change-Id: If9

MB-27121: [3/3] Plumb GET_MANIFEST opcode into engines

Add the opcode and required functions so that we can accept and
process a collections get manifest command.

Change-Id: If9be0721d55c6bc233290e370f7c599f1a502fc2
Reviewed-on: http://review.couchbase.org/86442
Reviewed-by: Trond Norbye <trond.norbye@gmail.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


# 9628a0d6 07-Dec-2017 Jim Walker <jim@couchbase.com>

MB-27122: [2/2] Do not initialise the current manifest

As described in the MB, it is risky to initialise the current manifest
to a 'default' state. It is safer to run without a manifest

MB-27122: [2/2] Do not initialise the current manifest

As described in the MB, it is risky to initialise the current manifest
to a 'default' state. It is safer to run without a manifest as we
really should only operate on what the cluster_manager tells us.

This leads to one situation in DcpOpen where we must fail the open
if the manifest hasn't yet been set, this situation is now flagged
by a new error code "no_collections_manifest" allowing what should
be a temporary failure (cluster manager should always push a manifest)
to be observed.

Change-Id: I79804bbc97da6ba00182355815abe53ad1125da1
Reviewed-on: http://review.couchbase.org/86514
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


# d0d71f8a 07-Dec-2017 Jim Walker <jim@couchbase.com>

MB-27122: [1/2] Change DcpProducer filter from pointer to object

1. A DcpProducer can never not have a filter so make it an owned
object.

2. In-order to get better error return

MB-27122: [1/2] Change DcpProducer filter from pointer to object

1. A DcpProducer can never not have a filter so make it an owned
object.

2. In-order to get better error returns for incorrect filters on
DcpOpen hoist the construction of the filter to be part of the
producer creation. We now create the filter and if success create
the producer passing the filter as an argument, this is done using
std::move so that we don't create temporaries.

3. The error checking is now geared around catching cb::engine_error
allowing the creation to return errors to the client. Note at this
patch level the Filter's throw points are unchanged, it still throws
std::invalid_argument and triggers a disconnect.

Change-Id: Ife88598830dcaf27573783228c989dcc6a31a9bc
Reviewed-on: http://review.couchbase.org/86513
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


# 6e741613 28-Nov-2017 Jim Walker <jim@couchbase.com>

MB-16181: Make collection size be configurable

Current PRD stated 1,000 collections per bucket maximum, so enforce
this using a config param - "collections_max_size"

Change-Id:

MB-16181: Make collection size be configurable

Current PRD stated 1,000 collections per bucket maximum, so enforce
this using a config param - "collections_max_size"

Change-Id: I482d96a806539aaadf0a045a21b49f398bc899d3
Reviewed-on: http://review.couchbase.org/86000
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


Revision tags: v5.1.0, v5.0.0
# 30aab6b7 22-Aug-2017 Jim Walker <jim@couchbase.com>

MB-25342: Collection UID in the Manifest

Primarily update the JSON specification so that the 'set_collections'
command requires that collections are given a UID.

The purpose of

MB-25342: Collection UID in the Manifest

Primarily update the JSON specification so that the 'set_collections'
command requires that collections are given a UID.

The purpose of the UID is to allow KV-engine to see that a collection
was deleted and recreated without ever seeing the intermediate delete.
The collections management code will identify that it knows about a
collection, but because of the UID it can determine if the collection
it knows about is now defunct.

A secondary purpose is that DCP may allow clients to use UIDs when
filtering collections.

The UID must be a JSON string that stores a 64-bit unsigned integer
encoded in hex.

Note it is the cluster managements responsibility to ensure that a UID
is unique enough that it doesn't collide with a previous generation
before that generation has been fully deleted (note once a generation
has been fully deleted, a UID could be re-used).

Change-Id: I6f9bbde00783a7a93511bb5ee426b8c43d8c019f
Reviewed-on: http://review.couchbase.org/83227
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


Revision tags: v4.6.2_ep
# 9e4dcf1f 23-Mar-2017 Jim Walker <jim@couchbase.com>

MB-16181: Add more ostream operators and dump methods

Add more ostream operators to the Collections classes and also give
them dump() methods so we can easily trigger output to stderr.

MB-16181: Add more ostream operators and dump methods

Add more ostream operators to the Collections classes and also give
them dump() methods so we can easily trigger output to stderr.

Finally add a hook so we can force the logging of a lot of collection
state, this will eventually be migrated to stats and more supported
debug methods.

Change-Id: I18abe41cd8f00fe18cb4fb972fcc054375f02d39
Reviewed-on: http://review.couchbase.org/78639
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


# 14fcb066 15-Mar-2017 Jim Walker <jim@couchbase.com>

MB-16181: Integrate filters into DcpProducer and ActiveStream

The DCP producer now adds a filter to itself when opened, the filter
is configured from the DCP open input data.

Op

MB-16181: Integrate filters into DcpProducer and ActiveStream

The DCP producer now adds a filter to itself when opened, the filter
is configured from the DCP open input data.

Opening DCP without DCP_OPEN_COLLECTIONS allows access to the
default collection only (if it exists).

Opening DCP with DCP_OPEN_COLLECTIONS allows the client to specify
a filter or if no-value is given, a passthrough filter is created.

Change-Id: I6a8c24c0af20a07ddd029325487172b516e54b00
Reviewed-on: http://review.couchbase.org/78637
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


# ef22f9b0 25-May-2017 Dave Rigby <daver@couchbase.com>

Move ep-engine to engines/ep


# 126ed5a4 10-Mar-2017 Jim Walker <jim@couchbase.com>

MB-16181: Add collections.set_manifest support

Add a method which will accept the new manifest and apply it to
all active vbuckets.

The latest manifest is saved in memory and al

MB-16181: Add collections.set_manifest support

Add a method which will accept the new manifest and apply it to
all active vbuckets.

The latest manifest is saved in memory and also used for when any VB
is set to active

Change-Id: Ic6a339bc5af279d105b679f528ff3675d1f16ac7
Reviewed-on: http://review.couchbase.org/77436
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...