Skip to content
This repository has been archived by the owner on Feb 20, 2021. It is now read-only.

Commit

Permalink
multi bulk requests in redis-benchmark, default fsync policy changed …
Browse files Browse the repository at this point in the history
…to everysec, added a prefix character for DEBUG logs
  • Loading branch information
antirez committed Feb 6, 2010
1 parent 3c290b9 commit 6766f45
Show file tree
Hide file tree
Showing 4 changed files with 36 additions and 21 deletions.
8 changes: 5 additions & 3 deletions TODO
@@ -1,7 +1,7 @@
Redis TODO and Roadmap

VERSION 1.4 TODO (Hash type)
============================
VERSION 2.0 TODO
================

* BRPOPLPUSH
* List ops like L/RPUSH L/RPOP should return the new list length.
Expand All @@ -21,7 +21,9 @@ Virtual Memory sub-TODO:
* Check what happens performance-wise if instead to create threads again and again the same threads are reused forever. Note: this requires a way to disable this clients in the child, but waiting for empty new jobs queue can be enough.
* Sets of integers are slow to load, for a number of reasons. Fix it. (use slow_sets.rdb file for debugging).

* Hashes (GET/SET/DEL/INCRBY/EXISTS/FIELDS/LEN/MSET/MGET). Special encoding for hashes with < N keys.
* Hashes (GET/SET/DEL/INCRBY/EXISTS/FIELDS/LEN/MSET/MGET). Special encoding for hashes with less than N elements.
* Write documentation for APPEND
* Implement LEN, SUBSTR, PEEK, POKE, SETBIT, GETBIT

VERSION 2.2 TODO (Fault tolerant sharding)
===========================================
Expand Down
27 changes: 18 additions & 9 deletions redis-benchmark.c
Expand Up @@ -525,6 +525,24 @@ int main(int argc, char **argv) {
}

do {
prepareForBenchmark();
c = createClient();
if (!c) exit(1);
c->obuf = sdscat(c->obuf,"PING\r\n");
prepareClientForReply(c,REPLY_RETCODE);
createMissingClients(c);
aeMain(config.el);
endBenchmark("PING");

prepareForBenchmark();
c = createClient();
if (!c) exit(1);
c->obuf = sdscat(c->obuf,"*1\r\n$4\r\nPING\r\n");
prepareClientForReply(c,REPLY_RETCODE);
createMissingClients(c);
aeMain(config.el);
endBenchmark("PING (multi bulk)");

prepareForBenchmark();
c = createClient();
if (!c) exit(1);
Expand Down Expand Up @@ -577,15 +595,6 @@ int main(int argc, char **argv) {
aeMain(config.el);
endBenchmark("LPOP");

prepareForBenchmark();
c = createClient();
if (!c) exit(1);
c->obuf = sdscat(c->obuf,"PING\r\n");
prepareClientForReply(c,REPLY_RETCODE);
createMissingClients(c);
aeMain(config.el);
endBenchmark("PING");

prepareForBenchmark();
c = createClient();
if (!c) exit(1);
Expand Down
2 changes: 1 addition & 1 deletion redis.c
Expand Up @@ -873,7 +873,7 @@ static void redisLog(int level, const char *fmt, ...) {

va_start(ap, fmt);
if (level >= server.verbosity) {
char *c = ".-*";
char *c = ".-*#";
char buf[64];
time_t now;

Expand Down
20 changes: 12 additions & 8 deletions redis.conf
Expand Up @@ -158,14 +158,18 @@ appendonly no
# always: fsync after every write to the append only log . Slow, Safest.
# everysec: fsync only if one second passed since the last fsync. Compromise.
#
# The default is "always" that's the safer of the options. It's up to you to
# understand if you can relax this to "everysec" that will fsync every second
# or to "no" that will let the operating system flush the output buffer when
# it want, for better performances (but if you can live with the idea of
# some data loss consider the default persistence mode that's snapshotting).

appendfsync always
# appendfsync everysec
# The default is "everysec" that's usually the right compromise between
# speed and data safety. It's up to you to understand if you can relax this to
# "no" that will will let the operating system flush the output buffer when
# it wants, for better performances (but if you can live with the idea of
# some data loss consider the default persistence mode that's snapshotting),
# or on the contrary, use "always" that's very slow but a bit safer than
# everysec.
#
# If unsure, use "everysec".

# appendfsync always
appendfsync everysec
# appendfsync no

################################ VIRTUAL MEMORY ###############################
Expand Down

0 comments on commit 6766f45

Please sign in to comment.