bench: return EXIT_FAILURE if an error occurs

Change-Id: I4cf80a230248c35d1d7e37ebb7ddb61a1a9cae7f
Reviewed-on: http://review.couchbase.org/75691
Tested-by: Build Bot <build@couchbas

bench: return EXIT_FAILURE if an error occurs

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

show more ...

MB-20199: GET_COUNT: Count the size of objects

Change-Id: I8b9d25e66abccb91c229c916b47a543f8cb60000
Reviewed-on: http://review.couchbase.org/65902
Reviewed-by: Dave Rigby <daver@couc

MB-20199: GET_COUNT: Count the size of objects

Change-Id: I8b9d25e66abccb91c229c916b47a543f8cb60000
Reviewed-on: http://review.couchbase.org/65902
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: buildbot <build@couchbase.com>

show more ...

Mute warnings about array subscript

warning: array subscript is of type 'char'

The datatype char is signed and may result in a negative
offset.

Change-Id: Ifc285510e129

Mute warnings about array subscript

warning: array subscript is of type 'char'

The datatype char is signed and may result in a negative
offset.

Change-Id: Ifc285510e12999e1e6122d8a0e044594d1f38d90
Reviewed-on: http://review.couchbase.org/54345
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

bench: allow 'raw' mode.

This mode populates a character table and iterates over the input,
attempting to check if the character matches any in the table. Compare
with actual JSON pr

bench: allow 'raw' mode.

This mode populates a character table and iterates over the input,
attempting to check if the character matches any in the table. Compare
with actual JSON processing

Change-Id: I39313a70e5f715367b5658652e935e31fed0952a
Reviewed-on: http://review.couchbase.org/53260
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Use string inputs/outputs for counter operations

- Remove INCREMENT and DECREMENT commands. These are merged into a
single COUNTER command
- Remove set_delta. All commands now acce

Use string inputs/outputs for counter operations

- Remove INCREMENT and DECREMENT commands. These are merged into a
single COUNTER command
- Remove set_delta. All commands now accept strings
- Remove Result::numresult(). Use string result.

Change-Id: I43d018074cfd45a64c5a9657ee2f330fe40d7063
Reviewed-on: http://review.couchbase.org/51667
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Remove "built-in" result object

All subdoc operations must now operate on an external result

Change-Id: Ic17dc3ab37a021862a1d2a3f5cc464701135bf8d
Reviewed-on: http://review.couc

Remove "built-in" result object

All subdoc operations must now operate on an external result

Change-Id: Ic17dc3ab37a021862a1d2a3f5cc464701135bf8d
Reviewed-on: http://review.couchbase.org/51339
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: buildbot <build@couchbase.com>

show more ...

Create Result output class

This class is intended to store buffers/operation-specific data on a
per-execution basis, so that the Operation structure can largely remain
itself statele

Create Result output class

This class is intended to store buffers/operation-specific data on a
per-execution basis, so that the Operation structure can largely remain
itself stateless. This allows result buffers to persist passed a
subsequent invocation of Operation::op_exec()

Change-Id: If2d95ec27c9f05039211f8e625b1c660bcdff839
Reviewed-on: http://review.couchbase.org/51285
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

bench: use %f instead of %lf

While %lf is legal, it's redundant.

Change-Id: Ic97fbe10c050846dd389d5c6d4f9de6d20d6c3a4
Reviewed-on: http://review.couchbase.org/51282
Tested-b

bench: use %f instead of %lf

While %lf is legal, it's redundant.

Change-Id: Ic97fbe10c050846dd389d5c6d4f9de6d20d6c3a4
Reviewed-on: http://review.couchbase.org/51282
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Make Operation fields private (2/2)

Change-Id: I6b2a7beedecbb72b72c2749b39ca2c3856ae141b
Reviewed-on: http://review.couchbase.org/51109
Reviewed-by: Dave Rigby <daver@couchbase.com>

Make Operation fields private (2/2)

Change-Id: I6b2a7beedecbb72b72c2749b39ca2c3856ae141b
Reviewed-on: http://review.couchbase.org/51109
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: buildbot <build@couchbase.com>

show more ...

bench: remove usage of old "C-style" names

Change-Id: Ic474410024986a6a6874129c003089a545ead322
Reviewed-on: http://review.couchbase.org/51061
Tested-by: buildbot <build@couchbase.co

bench: remove usage of old "C-style" names

Change-Id: Ic474410024986a6a6874129c003089a545ead322
Reviewed-on: http://review.couchbase.org/51061
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

bench: use std::chrono

Change-Id: Ia8ec3ebc3751bd20b4eb8382e863a905d9b60026
Reviewed-on: http://review.couchbase.org/51060
Tested-by: buildbot <build@couchbase.com>
Reviewed-by:

bench: use std::chrono

Change-Id: Ia8ec3ebc3751bd20b4eb8382e863a905d9b60026
Reviewed-on: http://review.couchbase.org/51060
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Mark Nunberg <mark.nunberg@couchbase.com>

show more ...

Refactor numeric ops

- Provide dedicated Operation::set_delta (no longer needed to ntohll)
- Provide accessor for output result

Change-Id: I45f7bcb6d21c86e54b08c5b72de41824ecf9f

Refactor numeric ops

- Provide dedicated Operation::set_delta (no longer needed to ntohll)
- Provide accessor for output result

Change-Id: I45f7bcb6d21c86e54b08c5b72de41824ecf9f0f8
Reviewed-on: http://review.couchbase.org/51059
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Add Command::ARRAY_INSERT

This allows insertion operations as specified in the spec.

Change-Id: I667df1d7f66de147823e12a4fbbddff0d8e62ee9
Reviewed-on: http://review.couchbase.or

Add Command::ARRAY_INSERT

This allows insertion operations as specified in the spec.

Change-Id: I667df1d7f66de147823e12a4fbbddff0d8e62ee9
Reviewed-on: http://review.couchbase.org/50798
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Add LICENSE and copyright headers

Change-Id: Ibc9fcf211a2ad6ba68918bc0f6a2d494d34d8b47
Reviewed-on: http://review.couchbase.org/50796
Tested-by: buildbot <build@couchbase.com>
Re

Add LICENSE and copyright headers

Change-Id: Ibc9fcf211a2ad6ba68918bc0f6a2d494d34d8b47
Reviewed-on: http://review.couchbase.org/50796
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Convert error/command codes to namespaced/scoped versions (v2)

Definitions are still provided for older code

This is a re-reversion of a prior commit which fixes some other issues

Convert error/command codes to namespaced/scoped versions (v2)

Definitions are still provided for older code

This is a re-reversion of a prior commit which fixes some other issues
with build failures, and adds a bit more as well.

Change-Id: I8105a55e3f100fc9011ab04ca3482cf464d395db
Reviewed-on: http://review.couchbase.org/50795
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Revert "Convert error/command codes to namespaced/scoped versions"

This reverts commit 50dab6a66df2390f56dbb417aeafbe656e0f07ef.

Change-Id: Ib97d092544afe1ee3fe899b78be36de5e10c5837

Revert "Convert error/command codes to namespaced/scoped versions"

This reverts commit 50dab6a66df2390f56dbb417aeafbe656e0f07ef.

Change-Id: Ib97d092544afe1ee3fe899b78be36de5e10c5837
Reviewed-on: http://review.couchbase.org/50765
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Dave Rigby <daver@couchbase.com>

show more ...

Convert error/command codes to namespaced/scoped versions

Definitions are still provided for older code

Change-Id: I255f9431aec3750d89e604388d5da8c0c099f3e2
Reviewed-on: http://

Convert error/command codes to namespaced/scoped versions

Definitions are still provided for older code

Change-Id: I255f9431aec3750d89e604388d5da8c0c099f3e2
Reviewed-on: http://review.couchbase.org/50759
Reviewed-by: Mark Nunberg <mark.nunberg@couchbase.com>
Tested-by: Mark Nunberg <mark.nunberg@couchbase.com>

show more ...

Remove C-style wrappers (1/2)

This adds some C++ methods and refactors the tests to use these new
methods rather than the old C-style wrappers.

Change-Id: Ifcc4042e6fd59ec3ec9b7

Remove C-style wrappers (1/2)

This adds some C++ methods and refactors the tests to use these new
methods rather than the old C-style wrappers.

Change-Id: Ifcc4042e6fd59ec3ec9b7c4a0ea176879c9efea0
Reviewed-on: http://review.couchbase.org/50602
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Migrate "operations.cc" to slightly more idiomatic C++

This removes 'subdoc_STRING' and uses an std::string underneath. The C++
API is still abstracted via C wrappers (which are themselv

Migrate "operations.cc" to slightly more idiomatic C++

This removes 'subdoc_STRING' and uses an std::string underneath. The C++
API is still abstracted via C wrappers (which are themselves
lightweight)

Change-Id: I0373752e34b790d5f4652198343282325b82feab
Reviewed-on: http://review.couchbase.org/50124
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Use correct type (subdoc_ERRORS) for status codes

Instead of a generic uint16_t, use the correct type (subdoc_ERRORS)
for subjson status codes.

Use correct type (subdoc_OPTYPE) for opcodes

Instead of a generic uint8_t, use the correct type (subdoc_OPTYPE) for
opcode variables.

Update to be C++11 compliant

C++11 is stricter about running literals together:

error: invalid suffix on literal; C++11 requires a space between literal and identifier [-Wreserv

Update to be C++11 compliant

C++11 is stricter about running literals together:

error: invalid suffix on literal; C++11 requires a space between literal and identifier [-Wreserved-user-defined-literal]
n_buf = sprintf(op->numbufs, "%"PRId64, num_i);

show more ...

Fix extra '--' in '--create-intermediate'

bench: Improve help listings and remove _p commands

A new 'help' command will list the command.
The -M option will have the effect of _p

Remove PUSH/POP