summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* RDB: try to make error handling code more readable.diskless-short-readantirez2019-07-173-47/+52
|
* Rio: when in error condition avoid doing the operation.antirez2019-07-171-0/+2
|
* RDB: modify rdbReportError() var name for clarity.antirez2019-07-171-4/+4
|
* Rio: remember read/write error conditions.antirez2019-07-173-6/+32
|
* prevent diskless replica from terminating on short readOran Agra2019-07-176-68/+250
| | | | | | | | now that replica can read rdb directly from the socket, it should avoid exiting on short read and instead try to re-sync. this commit tries to have minimal effects on non-diskless rdb reading. and includes a test that tries to trigger this scenario on various read cases.
* Merge branch 'unstable' of github.com:/antirez/redis into unstableantirez2019-07-174-4/+16
|\
| * Merge pull request #6185 from AlexRenCN/unstableSalvatore Sanfilippo2019-07-161-1/+1
| |\ | | | | | | fix readme.md,Redis data types should add `t_stream.c`.
| | * fix readme.md,Redis data types should add `t_stream.c`.zheng.ren01@mljr.com2019-06-251-1/+1
| | |
| * | Merge pull request #5779 from madolson/dev-unstable-geo-bugfixSalvatore Sanfilippo2019-07-151-1/+13
| |\ \ | | | | | | | | Fixed a rounding bug in geo.tcl
| | * | Fixed a rounding bug in geo.tclMadelyn Olson2019-01-151-1/+13
| | | |
| * | | Merge pull request #6196 from AngusP/resp3Salvatore Sanfilippo2019-07-121-1/+1
| |\ \ \ | | | | | | | | | | [RESP3, Minor] in networking.c double representation for -infiinity leaves out comma
| | * | | RESP3 double representation for -infinity is `,-inf\r\n`, not `-inf\r\n`Angus Pearson2019-07-021-1/+1
| | | |/ | | |/|
| * | | Merge pull request #5738 from bmerry/fix-swapdb-int-overflowSalvatore Sanfilippo2019-07-121-1/+1
| |\ \ \ | | | | | | | | | | Make dbSwapDatabases take args as long
| | * | | Make dbSwapDatabases take args as longBruce Merry2018-12-311-1/+1
| | | | | | | | | | | | | | | | | | | | This prevents an integer overflow bug. Closes #5737.
* | | | | Vertically compact code in aofWrite().antirez2019-07-121-3/+1
|/ / / /
* | | | Merge branch 'unstable' of github.com:/antirez/redis into unstableantirez2019-07-101-1/+3
|\ \ \ \
| * \ \ \ Merge pull request #6198 from oranagra/skiplist-mem-usageSalvatore Sanfilippo2019-07-101-1/+3
| |\ \ \ \ | | | | | | | | | | | | missing per-skiplist overheads in MEMORY USAGE
| | * | | | missing per-skiplist overheads in MEMORY USAGEOran Agra2019-07-041-1/+3
| | | |/ / | | |/| | | | | | | | | | | | | | | | | these had severe impact for small zsets, for instance ones with just one element that is longer than 64 (causing it not to be ziplist encoded)
* | | | | Client side caching: filter clients no longer there.antirez2019-07-101-0/+1
|/ / / /
* | | | Client side caching: implement CLIENT GETREDIR.antirez2019-07-101-14/+22
| | | | | | | | | | | | | | | | | | | | | | | | This subcommand may simplify the writing of Redis client libraries using the tracking feature and/or improve observability and debugging capabilities.
* | | | Client side caching: add tracking clients in INFO.antirez2019-07-101-2/+4
| | | |
* | | | Diskless replica: fix disklessLoadRestoreBackups() bug.antirez2019-07-101-7/+7
| | | |
* | | | Diskless replica: refactoring of DBs backups.antirez2019-07-101-22/+47
| | | |
* | | | Diskless replica: fix mispelled var name.antirez2019-07-101-1/+1
| | | |
* | | | Diskless replica: a few aesthetic changes to rio.cantirez2019-07-081-25/+32
| | | |
* | | | Diskless replica: a few aesthetic changes to replication.c.antirez2019-07-082-42/+96
| | | |
* | | | Merge pull request #6193 from oranagra/diskless_slave_refresh3Salvatore Sanfilippo2019-07-0817-251/+647
|\ \ \ \ | | | | | | | | | | diskless replication on slave side (don't store rdb to file), plus some other related fixes
| * | | | diskless replication on slave side (don't store rdb to file), plus some ↵Oran Agra2019-07-0817-251/+647
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | other related fixes The implementation of the diskless replication was currently diskless only on the master side. The slave side was still storing the received rdb file to the disk before loading it back in and parsing it. This commit adds two modes to load rdb directly from socket: 1) when-empty 2) using "swapdb" the third mode of using diskless slave by flushdb is risky and currently not included. other changes: -------------- distinguish between aof configuration and state so that we can re-enable aof only when sync eventually succeeds (and not when exiting from readSyncBulkPayload after a failed attempt) also a CONFIG GET and INFO during rdb loading would have lied When loading rdb from the network, don't kill the server on short read (that can be a network error) Fix rdb check when performed on preamble AOF tests: run replication tests for diskless slave too make replication test a bit more aggressive Add test for diskless load swapdb
* | | | Merge pull request #6116 from AngusP/scan-typesSalvatore Sanfilippo2019-07-083-7/+72
|\ \ \ \ | | | | | | | | | | SCAN: New Feature `SCAN cursor [TYPE type]` modifier suggested in issue #6107
| * | | | Change typeNameCanonicalize -> getObjectTypeName, and other style changesAngus Pearson2019-07-082-8/+7
| | | | |
| * | | | Spelling cannonical -> canonicalAngus Pearson2019-06-131-1/+1
| | | | |
| * | | | Add char* typeNameCanonicalize(robj*) to remove duplicate code between SCAN ↵Angus Pearson2019-06-102-26/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and TYPE commands, and to keep OBJ_* enum to string canonicalization in one place.
| * | | | Implement `SCAN cursor [TYPE type]` modifier suggested in issue #6107.Angus Pearson2019-05-222-1/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add tests to check basic functionality of this optional keyword, and also tested with a module (redisgraph). Checked quickly with valgrind, no issues. Copies name the type name canonicalisation code from `typeCommand`, perhaps this would be better factored out to prevent the two diverging and both needing to be edited to add new `OBJ_*` types, but this is a little fiddly with C strings. The [redis-doc](https://github.com/antirez/redis-doc/blob/master/commands.json) repo will need to be updated with this new arg if accepted. A quirk to be aware of here is that the GEO commands are backed by zsets not their own type, so they're not distinguishable from other zsets. Additionally, for sparse types this has the same behaviour as `MATCH` in that it may return many empty results before giving something, even for large `COUNT`s.
* | | | | Merge pull request #6210 from gkorland/patch-3Salvatore Sanfilippo2019-07-071-1/+1
|\ \ \ \ \ | | | | | | | | | | | | fix build tracking.c should be tracking.o
| * | | | | fix build tracking.c should be tracking.oGuy Korland2019-07-071-1/+1
|/ / / / / | | | | | | | | | | thanks to @rafie
* | | | | Merge pull request #6162 from madolson/dev-unstable-config-refactorSalvatore Sanfilippo2019-07-071-265/+89
|\ \ \ \ \ | | | | | | | | | | | | Refactored yesno configs so there is less duplication
| * | | | | Refactored yesno configs so there was less duplicationMadelyn Olson2019-06-131-265/+89
| | | | | |
* | | | | | Merge pull request #6161 from swilly22/redismodule-loading-ctx-flagSalvatore Sanfilippo2019-07-072-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Extend REDISMODULE_CTX_FLAGS to indicate if redis is loading
| * | | | | | Extend REDISMODULE_CTX_FLAGS to indicate if redis is currently loading from ↵swilly222019-06-122-0/+5
| | |_|/ / / | |/| | | | | | | | | | | | | | | | either RDB or AOF
* | | | | | Merge pull request #6022 from itamarhaber/RedisModule_ReplyWithCStringSalvatore Sanfilippo2019-07-052-0/+13
|\ \ \ \ \ \ | | | | | | | | | | | | | | Adds RedisModule_ReplyWithCString
| * | | | | | Uses addReplyBulkCStringItamar Haber2019-04-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Itamar Haber <itamar@redislabs.com>
| * | | | | | Adds RedisModule_ReplyWithCStringItamar Haber2019-04-162-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Itamar Haber <itamar@redislabs.com>
* | | | | | | Client side caching: RESP2 support.antirez2019-07-052-2/+15
| | | | | | |
* | | | | | | Merge branch 'unstable' of github.com:/antirez/redis into unstableantirez2019-07-043-14/+18
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | |
| * | | | | | Merge pull request #6149 from madolson/acl-spellingSalvatore Sanfilippo2019-06-072-12/+12
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Fixed some spelling issues in ACL codepath including a user facing error
| | * | | | | | Fixed some spelling issues in ACL codepath including user facing errorMadelyn Olson2019-06-072-12/+12
| | | |/ / / / | | |/| | | |
| * | | | | | Merge pull request #6148 from artix75/redis_bm_devSalvatore Sanfilippo2019-06-051-2/+6
| |\ \ \ \ \ \ | | |_|_|/ / / | |/| | | | | Redis Benchmark: prevent CONFIG failure from exiting program
| | * | | | | Redis Benchmark: prevent CONFIG failure from exiting programartix2019-06-051-2/+6
| |/ / / / /
* | | | | | Client side caching: fix invalidate message len and content.antirez2019-07-041-2/+2
| | | | | |
* | | | | | Client side caching: implement trackingInvalidateKey().antirez2019-07-036-2/+170
| | | | | |