* Add test on usage run_triggers inside before_replace.
Test verifies that other before_replace triggers within the
same space will be executed but other triggers won't.
* Add test on return old in before_replace.
Test verifies that other before_replace triggers will be executed,
but insertion won't take place and other triggers will be ignored.

Closes #3128

Fix problem: cannot connect to unix binary socket using authentication.
To solve this issue update our uri parser to support following schema:
 login:password@unix/:/path1/path2/path3
Add tests

Closes: #2933

If curl does not have SSL support, don't panic, honor CURL_FIND_REQUIRED
variable, and panic only if it's set.

Correct the grammar in the error message.

On MAC several old curl versions don't support SSL by default.
Try to check it and print error message if so.

Close #3065

Closes #2666

When a field type is specified as numbered key, and a name is
named key, then the type is ignored. For example:
{name = '<name>', '<type>'} - here the '<type>' is ignored and
result space format contains 'any' regardless of <type>.

Fix tuple format field parsing to take this case into account.

Closes #2895

ERRINJ_VY_READ_PAGE and ERRINJ_VY_READ_PAGE_TIMEOUT injections are
used by vinyl/errinj test to check that a page read error is handled
properly. The cache checks if these injections are enabled and bails
out if so. Since commit a31c2c10 ("vinyl: force read view in
iterator in autocommit mode"), this is not necessary, because cache
is not used unless SELECT is called from a transaction, and the above
mentioned test doesn't use transactions. So let's remove the checks.
If we ever enable cache for all SELECTs, we can disable cache in the
test with box.cfg.vinyl_cache instead of using error injections.

Currently, new entries are added to the cache even if the limit is set
to 0, which is obviously incorrect. Fix it. This might be useful for
debugging and/or performance evaluation.

Closes #3172

Make box.cfg.vinyl_cache option dynamically configured. If the new
value is less than the current cache usage, it will block until it
reclaims enough cache entries to fit in the new limit.

If the user terminates tarantool while compaction is in progress,
tarantool won't exit immediately - it will hang until all dump and
compaction tasks that are currently in progress are complete, which
may take quite a while on large data sets. What is especially funny,
once a task has been finished, vinyl will not commit the produced run
file in the vylog, because the event loop has already been stopped,
and so will delete it after restart and schedule the task anew.

This patch makes the scheduler forcefully abort all running tasks
as soon as possible.

Closes #3166

This is pointless as worker threads don't use coio.

vy_run_writer incapsulates logic of writing statements to a run.
It provides API to write statements one by one. It is needed so
that we can abort a run writing task before waiting for it to
finish writing the file.

Edited by @locker:
 - add region_truncate() wherever necessary
 - do not reallocate row index ibuf for each page
 - remove vy_run_write(), use vy_run_writer directly instead

Needed for #3166

If a replica receives some changes then corresponding wal events are
generated and relay fiber is waken up before heartbeat timeout.
But there may be nothing to send if all changes are from the current
relay peer. In this case an applier doesn't receive anything and break a
connection.

Fixes #3160

Add memory occupied by cached slabs to the size reported by
arena_size and arena_used_ratio respectively.

This should make memory monitoring easier:

if items_used_ratio > 0.9 and arena_used_ratio > 0.9 and quota_used_ratio > 0.9 then
    we run out of memory
end

On FreeBSD we have either system iconv (part of libc) or the one
in /usr/local. The first one is not fully compatible with Linux/
OSX iconv and both adds specific prefixes to function names.
So to use them via Lua FFI mechanism, specific names are required.
Proposal fix:
Under FreeBSD link with /usr/local/lib/libiconv.so and add prefix.

Closes gh-3073.

* Add parser of cookies from 'Set-Cookie' header.
* All cookies are stored in response object in cookie table.
* The format of each table element is following:
resp[key] = {value, {option, ...}},
where key, value is cookie key, value
options are array of strings, set-cookie header options,
e.g. HttpOnly, Expires=<date>.

Closes #2801

* delete old small parser with nginx tested one
* functionality is not changed

If a key is updated after a secondary index scan, but before a
primary index lookup, then ignore this update.

Closes #2442

If a column is nullable and is the last defined one (via index parts
or space format), it can be omited on insertion. Such absent fields
are treated as NULLs in comparators and are not stored.

Closes #2988

This template parameter serves to detect, if two sequences of
key parts are actually parts of tuples. In such a case
key_compare_parts does additional comparison by primary key parts,
if a NULL was met.

This parameters is actually a crutch, put by me to avoid code
doubling in tuple_compare_sequential_nullable. And this parameter
complicates #2988, where a sequential nullable tuple contains
optional parts.

Needed for #2988

Closes #2048

They have no sense without key_def.h.

Needed for #2048

Update comments and move space_def_check_compatibility to
space_def.[hc].

On a single field two restrictions exist: tuple format and index part.

The proposal is to allow to use different types for different restrictions,
if these types are compatible. And for tuples validation use the most
strict type.

For example, one field can be number in space format, unsigned in one index,
integer in another index and scalar in a third index. Configuration like this
can be allowed, because all of these types can store unsigned. And unsigned is
used to validate tuples.

Besides, it allows incremental DDL. For example, now if you want to change a
type of a field, which is used by several indexes, then it is impossible on
a non-empty space, even if a new type is less strict than the old one.
Lets allow this too.

Closes #3008

At first, now if a space is not empty, its format can not be
restricted by nullability removal, or by restriction of a field type.

The workaround was format removal and reset. Lets allow normal format
restriction followed by space format checking.

At second, now an index is rebuild on any change of its key definition.
But there is no sense to rebuild it, if uniqueness is turned off, part
field numbers and collations are not changed, because it means, that
only types was restricted or extended. So format checking is enough.

Needed for #3008

Compatibility must be commutative, but this function is not
commutative. It checks, that one type can store values of another
type, but not conversely.

Closes #2973

Not all workloads need bloom filters enabled for all indexes. Let's
allow to disable them on per-index basis by setting bloom_fpr to 1.
This will allow to save some memory if bloom filters are unused.

Closes #3138