Merge pull request #461 from schlenk/unstable

Replace some unnecessary calls to echo and cat in tests

New time limit for protocol desync test set to 30 seconds to reduce false positives.

Remove loadfile() access from the scripting engine.

Even inside #if 0 comments are comments.

Merge remote-tracking branch 'origin/unstable' into unstable

Limit memory used by big SLOWLOG entries.

Two limits are added:

1) Up to SLOWLOG_ENTRY_MAX_ARGV arguments are logged.
2) Up to SLOWLOG_ENTRY_MAX_STRING bytes per argument are logged.
3) slowlog-max-len is set to 128 by default (was 1024).

The number of remaining arguments / bytes is logged in the entry
so that the user can understand better the nature of the logged command.

Merge pull request #440 from ErikDubbelboer/spelling

Fixed some spelling errors in comments

Merge pull request #414 from chobits/unstable

fix typos in tests/assets/default.conf

README now makes clear that our support for solaris derived systems is "best effort".

redis.conf AOF section comments improved.

New tests related to scripts max execution time.

SHUTDOWN NOSAVE now can stop a non returning script. Issue #466.

Currenly not used code in dict.c commented out.

Tests for scripting PRNG.

redis-cli --bigkeys output modified to be simpler to read..

redis-cli --bigkeys

Test SDIFF with first set empty.

Test SINTER against same integer elements, but different set encoding.

Test SINTER with non existing key.

Added an SMOVE test where src and dest key are the same.

New hash fuzzing test.

Document mostly dead code in RPOPLPUSH implementation.

Explicit RPOP/LPOP tests.

Test LINSERT syntax error.

Test LINDEX out of range index.

Merge pull request #456 from pietern/unstable-mkopts

Persist Makefile flags (unstable)

Marginally cleaner lookupKeyByPattern() implementation.

just fieldobj itself as sentinel of the fact a field object is used or
not, instead of using the filed length, that may be confusing both for
people and for the compiler emitting a warning.

More robust maxclients test.

Two small fixes to maxclients handling.

1) Don't accept maxclients set to < 0
2) Allow maxclients < 1024, it is useful for testing.

Replace unnecessary calls to echo and cat

Tcl's exec can send data to stdout itself, no need to call cat/echo for
that usually.

Added test for SORT corner case: pattern ending with just "->".

lookupKeyByPattern() used by SORT GET/BY rewritten. Fixes issue #460.

lookupKeyByPattern() was implemented with a trick to speedup the lookup
process allocating two fake Redis obejcts on the stack. However now that
we propagate expires to the slave as DEL operations the lookup of the
key may result into a call to expireIfNeeded() having the stack
allocated object as argument, that may in turn use it to create the
protocol to send to the slave. But since this fake obejcts are
inherently read-only this is a problem.

As a side effect of this fix there are no longer size limits in the
pattern to be used with GET/BY option of SORT.

See https://github.com/antirez/redis/issues/460 for bug details.

Less false positives in maxclients test, hopefully.

Persist `make` settings and trigger rebuild if anything changes

Don't set flags recursively

Remove unused LIBS variable

First set defaults, then do composition

Question mark assignment is not needed

Ignore gcov/lcov artifacts

The lcov target shouldn't clean

This is not needed because every change in compiler/linker flags
triggers a cleanup.

Rename ADD_*FLAGS -> REDIS_*FLAGS, REDIS_*FLAGS -> FINAL_*FLAGS

This reflects that REDIS_*FLAGS will only be used for compilation of
Redis and not for its dependencies. Similarly, that FINAL_*FLAGS are
composed of other variables and holds the options that are finally
passed to the compiler and linker.

Merge branch 'strict.lua' into unstable

A few var names fixed in Makefile.

I modified it for error in a previous commit doing search & replace.

New test for scripting engine: DECR_IF_GT.

EVAL errors are more clear now.

Use Lua tostring() before concatenation.

Tests modified to match the new global protection implementation.

mt.declared is no longer needed.

Lua global protection can now be simpified becuase we no longer have the
global() function. It's useless to occupy memory with this table, it is
also not faster because the metamethods we use are only called when a
global object does not exist or we are trying to create it from a
script.

Stop access to global vars. Not configurable.

After considering the interaction between ability to delcare globals in
scripts using the 'global' function, and the complexities related to
hanlding replication and AOF in a sane way with globals AND ability to
turn protection On and Off, we reconsidered the design. The new design
makes clear that there is only one good way to write Redis scripts, that
is not using globals. In the rare cases state must be retained across
calls a Redis key can be used.

Globals protection global() function modified for speed and correctness.

Tests for lua globals protection.

Scripting: globals protection can now be switched on/off.

Print arch bits with redis-server -v

memtest.c: integer overflow fixed.

Make gcov fixed.

Makefile now introduces Redis-specific CFLAGS / LDFLAGS. Gcov target fixed. Added comments to describe how it works.

Merge pull request #449 from ErikDubbelboer/unstable

added explanation for the magic 511 backlog number

added explanation for the magic 511 backlog number

Makefile.dep updated.

make dep: redirect output to Makefile.dep.

Make inline functions rioRead/Write/Tell static. This fixes issue #447.

Macros ULONG_ONEZERO / ULONG_ZEROONE were inverted in #ifdef to test 32/64 bit arch.

Clean up Makefiles

Remove unused variables. Instead of overriding non-standard variables
such as ARCH and PROF, use standard variables CFLAGS and LDFLAGS to
override Makefile settings. Move dependencies generated by `make dep` to
a separate file.

Everything x86 is little endian

Comment typo fixed. Clusetr -> Cluster.

Check write(2) return value to avoid warnings, because in this context failing write is not critical.

Minor MIGRATE implementation simplification about ttl handling.

Version 2.9.7.

It is now possible to enable/disable RDB checksum computation from redis.conf or via CONFIG SET/GET. Also CONFIG SET support added for rdbcompression as well.

RDB files now embed a crc64 checksum. Version of RDB bumped to 5.

Fixed compilation of new rio.c changes (typos and so forth.)

dump/restore fixed to use the new crc64 API.

Add checksum computation to rio.c

crc64.c modified for incremental computation.

rio.c file somewhat documented so that the casual reader can understand what's going on without reading the code.

Test for maxclients.

Update src/rdb.c

Update src/dict.c

Fixed some spelling errors in the comments

Added new test to check that "CONFIG appendonly no" actually kills the background AOF operation in progress if any.

For coverage testing use exit() instead of _exit() when termiating saving children.

Tests for MONITOR.

New client info field added to CLIENT LIST output: multi, containing the length of the current pipeline. Test modified accordingly.

Never used function stringObjectEqualsMs() removed.

Removed dead code: function rdbSaveTime() is no longer used since RDB now saves expires in milliseconds.

Two new tests for BGREWRTIEAOF.

Check for scheduled rewrite if a BGSAVAE is in progress.
Check for error if a rewrite is already in progress.

redis.tcl: no longer leave unread replies if an error happens during a MULTI/EXEC block.

New INFO field in persistence section: bgrewriteaof_scheduled.

version bumped to 2.9.6

implement aeWait using poll(2). Fixes issue #267.

expireGenericCommand(): better variable names and a top-comment that describes the function's behavior.

for (p)expireat use absolute time, without double recomputation

fix mstime() ommited while comparing if key is already expired

add support for generation of lcov coverage reports

Merge pull request #430 from anydot/f-testsuite

Two changes of testsuite (killing redis and new option)

Merge pull request #426 from anydot/fix-rm-vm-comments

remove mentions of VM in comments

Merge pull request #431 from anydot/f-signal

allocate alternate signal stack, change of sigaction flags for sigterm

future-proof version comparison

On slow computers, 10 seconds are not enough for this heavy replication test.

Structure field controlling the INFO field master_link_down_since_seconds initialized correctly to avoid strange INFO output at startup when a slave has yet to connect to its master.