summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | aof: format code and commentzhaozhao.zz2018-01-151-5/+5
| | | | | | |
* | | | | | | Put more details in the comment introduced by #4601.antirez2018-01-151-3/+8
| | | | | | |
* | | | | | | Merge pull request #4601 from soloestoy/fix-memoryleak-for-lazy-server-delSalvatore Sanfilippo2018-01-151-3/+4
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | lazyfree: fix memory leak for lazyfree-lazy-server-del
| * | | | | | | lazyfree: fix memory leak for lazyfree-lazy-server-delzhaozhao.zz2018-01-151-3/+4
| | | | | | | |
* | | | | | | | Merge pull request #4575 from soloestoy/bugfix-benchmarkSalvatore Sanfilippo2018-01-121-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | redis-benchmark: bugfix - handle zero liveclients in right way
| * | | | | | | | redis-benchmark: bugfix - handle zero liveclients in right wayzhaozhao.zz2018-01-031-1/+1
| |/ / / / / / /
* | | | | | | | Merge pull request #4581 from dvirsky/module_unlinkSalvatore Sanfilippo2018-01-123-0/+56
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Added RM_UnlinkKey - a low level analog to UNLINK command
| * | | | | | | | Added RM_UnlinkKey - a low level analog to UNLINK commandDvir Volk2018-01-073-0/+56
| | | | | | | | |
* | | | | | | | | Merge branch 'unstable' of github.com:/antirez/redis into unstableantirez2018-01-121-1/+1
|\ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ Merge pull request #4586 from gnuhpc/fix-crashlog-typoSalvatore Sanfilippo2018-01-121-1/+1
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Fix a typo(maybe instruction?) in crash log
| | * | | | | | | | | Fix a typo(maybe instruction?) in crash loggnuhpc2018-01-091-1/+1
| | | | | | | | | | |
* | | | | | | | | | | Fix getKeysUsingCommandTable() in the case of nagative arity.antirez2018-01-121-5/+7
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a crash with Redis Cluster when OBJECT is mis-used, because getKeysUsingCommandTable() will call serverPanic() detecting we are accessing an invalid argument in the case "OBJECT foo" is called. This bug was introduced when OBJECT HELP was introduced, because the key argument is set fixed at index 2 in the command table, however now OBJECT may be called with an insufficient number of arguments to extract the key. The "Right Thing" would be to have a specific function to extract keys from the OBJECT command, however this is kinda of an overkill, so I preferred to make getKeysUsingCommandTable() more robust and just return no keys when it's not possible to honor the command table, because new commands are often added and also there are a number with an HELP subcommand violating the normal form, and crashing for this trivial reason or having many command-specific key extraction functions is not great.
* | | | | | | | | | Document new protocol options in #4568 into redis.conf.antirez2018-01-111-0/+14
| | | | | | | | | |
* | | | | | | | | | proto-max-querybuf-len -> client-query-buffer-limit.antirez2018-01-111-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We already had client buffer limits exported as configuration options. Stick with the naming scheme already used. See #4568.
* | | | | | | | | | New config options about protocol prefixed with "proto".antirez2018-01-114-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to #4568.
* | | | | | | | | | Merge pull request #4568 from oranagra/restore_size_limitSalvatore Sanfilippo2018-01-1111-34/+55
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | fix RESTORE command size limits
| * | | | | | | | | | Add config options for max-bulk-len and max-querybuf-len mainly to support ↵Oran Agra2017-12-294-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RESTORE of large keys
| * | | | | | | | | | fix processing of large bulks (above 2GB)Oran Agra2017-12-298-33/+39
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - protocol parsing (processMultibulkBuffer) was limitted to 32big positions in the buffer readQueryFromClient potential overflow - rioWriteBulkCount used int, although rioWriteBulkString gave it size_t - several places in sds.c that used int for string length or index. - bugfix in RM_SaveAuxField (return was 1 or -1 and not length) - RM_SaveStringBuffer was limitted to 32bit length
* | | | | | | | | | Merge pull request #3356 from yusaku/fix-module-firstkeySalvatore Sanfilippo2018-01-091-1/+1
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Fix the firstkey, lastkey, and keystep of moduleCommand
| * | | | | | | | | | Fix the firstkey, lastkey, and keystep of moduleCommandYusaku Kaneta2016-06-291-1/+1
| | | | | | | | | | |
* | | | | | | | | | | Test: MIGRATE AUTH test added.antirez2018-01-091-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to #2507.
* | | | | | | | | | | Rewrite MIGRATE AUTH option.antirez2018-01-091-12/+38
| |_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See PR #2507. This is a reimplementation of the fix that contained different problems.
* | | | | | | | | | Merge pull request #4579 from gnuhpc/fix-memoryusage-listSalvatore Sanfilippo2018-01-051-1/+1
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | | Fix memory usage list bug
| * | | | | | | | | Fix memory usage list buggnuhpc2018-01-051-1/+1
|/ / / / / / / / /
* | | | | | | | | Hyperloglog: refresh hdr variable correctly.antirez2017-12-221-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a fix for the #3819 improvements. The o->ptr may change because of hllSparseSet() calls, so 'hdr' must be correctly re-fetched.
* | | | | | | | | Hyperloglog: Support for PFMERGE sparse encoding as target.antirez2017-12-221-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a fix for #3819.
* | | | | | | | | Hyperloglog: refactoring of sparse/dense add function.antirez2017-12-221-20/+38
| |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The commit splits the add functions into a set() and add() set of functions, so that it's possible to set registers in an independent way just having the index and count. Related to #3819, otherwise a fix is not possible.
* | | | | | | | Merge pull request #4540 from hqin6/unstableSalvatore Sanfilippo2017-12-151-4/+6
|\ \ \ \ \ \ \ \ | | |/ / / / / / | |/| | | | | | fixbug for #4538 Error opening /setting AOF rewrite IPC pipes: No suc…
| * | | | | | | fixbug for #4538 Error opening /setting AOF rewrite IPC pipes: No such file ↵heqin2017-12-151-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | or directory
* | | | | | | | Merge pull request #4536 from tporadowski/antirez_unstable-cli-historySalvatore Sanfilippo2017-12-141-1/+2
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Always enable command history in redis-cli run on a TTY
| * | | | | | | always enable command history in redis-cliTomasz Poradowski2017-12-131-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - when redis-cli is running in a TTY - always enable command history buffering, regardless if history file path can be successfully determined
* | | | | | | | Merge pull request #4528 from nashe/patch-oob-read-4527Salvatore Sanfilippo2017-12-141-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Prevent off-by-one read in stringmatchlen()
| * | | | | | | | Prevent off-by-one read in stringmatchlen() (fixes #4527)nashe2017-12-121-1/+1
| | | | | | | | |
* | | | | | | | | safe_write -> aofWrite. Function commented.antirez2017-12-141-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to #4498.
* | | | | | | | | Merge pull request #4498 from soloestoy/aof-safe-writeSalvatore Sanfilippo2017-12-141-2/+23
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | | Aof safe write -- fix the short write
| * | | | | | | | aof: cast sdslen to ssize_tzhaozhao.zz2017-11-301-1/+1
| | | | | | | | |
| * | | | | | | | aof: fix the short writezhaozhao.zz2017-11-301-1/+22
| | | | | | | | |
* | | | | | | | | Cluster: allow read-only EVAL/EVALSHA in slaves.antirez2017-12-131-1/+2
| |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | Fix #3665.
* | | | | | | | Change indentation and other minor details of PR #4489.antirez2017-12-069-108/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The main change introduced by this commit is pretending that help arrays are more text than code, thus indenting them at level 0. This improves readability, and is an old practice when defining arrays of C strings describing text. Additionally a few useless return statements are removed, and the HELP subcommand capitalized when printed to the user.
* | | | | | | | C styleItamar Haber2017-12-051-4/+2
| | | | | | | |
* | | | | | | | Uses an offset in addReplyHelpItamar Haber2017-12-051-5/+3
| | | | | | | |
* | | | | | | | Merge remote-tracking branch 'upstream/unstable' into help_subcommandsItamar Haber2017-12-0545-376/+3316
|\ \ \ \ \ \ \ \
| * | | | | | | | add linkClient(): adds the client and caches the list node.antirez2017-12-053-8/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have this operation in two places: when caching the master and when linking a new client after the client creation. By having an API for this we avoid incurring in errors when modifying one of the two places forgetting the other. The function is also a good place where to document why we cache the linked list node. Related to #4497 and #4210.
| * | | | | | | | Merge pull request #4497 from soloestoy/optimize-unlink-clientSalvatore Sanfilippo2017-12-053-4/+12
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | networking: optimize unlinkClient() in freeClient()
| | * | | | | | | | networking: optimize unlinkClient() in freeClient()zhaozhao.zz2017-11-303-4/+12
| | |/ / / / / / /
| * | | | | | | | Merge pull request #4508 from trevor211/fixNotesSalvatore Sanfilippo2017-12-052-3/+6
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | fix some notes
| | * | | | | | | | fix some notesWuYunlong2017-12-052-3/+6
| | | | | | | | | |
| * | | | | | | | | Merge branch 'unstable' of github.com:/antirez/redis into unstableantirez2017-12-052-6/+1
| |\ \ \ \ \ \ \ \ \
| | * \ \ \ \ \ \ \ \ Merge pull request #4488 from itamarhaber/debug_aritySalvatore Sanfilippo2017-12-052-6/+1
| | |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Standardizes arity handling of DEBUG
| | | * | | | | | | | | Standardizes arity handling of DEBUGItamar Haber2017-11-282-6/+1
| | | | |/ / / / / / / | | | |/| | | | | | |