Commit Graph

389 Commits

Author SHA1 Message Date
antirez
f7f022e4e0 On stack buffer to read replies set to 16k (was 2k).
It was verified experimentally that this value, on Linux kernels, provides
better performances compared to the 2k value. However larger values
apparently don't produce any noticeable effect on performances.
2012-08-21 15:10:16 +02:00
antirez
51ab89d899 Max depth of multi-bulk reply moved from 2 to 7.
Hiredis can handle multi bulk replies with a fixed (hardcoded) level of
nesting. This should be changed in the future in order to avoid
hardcoded limits. As a quick fix this commit moves the max nesting from 2
to 7, so that there are no problems when processing replies from the SLOWLOG
command, from Redis Sentinel, or generated by Redis Lua Scripts (that are
allowed to generate replies with any level of nesting).
2012-08-21 15:06:06 +02:00
antirez
7f095053c6 Configurable reader max idle buffer size.
Hiredis used to free unused redisReader buffers bigger than 16k. Now
this limit is configurable (see the documentation updated by this commit)
in order to allow working with big payloads without incurring to speed
penalty.
2012-08-21 15:01:47 +02:00
antirez
7f346477c8 Add ARCH to Makefile CFLAGS / LDFLAGS.
This makes builiding with an optional 32 bit target simpler. For
instance Redis (that contains an embedded copy of hiredis) when compiled
with "make 32bit" uses to pass an ARCH parameter to force an hiredis
32 bit build.
2012-08-21 14:29:49 +02:00
Pieter Noordhuis
7ec4df9403 Spontaneous error reply can always happen 2012-07-31 17:40:46 -07:00
Pieter Noordhuis
f2ddeeae95 Merge pull request #101 from ErikDubbelboer/spelling
Fixed typo in README
2012-07-11 08:49:48 -07:00
Pieter Noordhuis
63e34f5fe0 Merge pull request #112 from nessence/master
MONITOR causes assertion failure, async.c, line 398.
2012-07-11 08:49:20 -07:00
Alex Leverington
01cce89d9d async: support for determining monitor mode, if so, repush replies callback in expectation of another reply. 2012-07-11 02:53:58 -05:00
Mark Ellzey
f8debbfdbe Use poll() instead of select() inside redisContextWaitReady()
The current select() is limiting in instances where the fd num is > FD_SETSIZE.
Since redisContextWaitReady() only processes a single fd, select would still
fail.

For compatibility reasons I have converted select() over to poll(), eliminating
this problem.
2012-04-18 12:18:07 -07:00
Erik Dubbelboer
bd2501078e Fixed typo in README 2012-04-07 15:07:40 +02:00
Pieter Noordhuis
857b2690af Update README with actual async callback behavior
See issue #79, thanks to @ewencp.
2012-02-06 11:08:39 -08:00
Pieter Noordhuis
d5d884378d Merge pull request #72 from chipdude/master
Fix self-test of connect error on Linux; merge duplicated string (DRY)
2011-12-20 10:45:27 -08:00
Chip Salzenberg
56ae8aa110 DRY list of integer formats 2011-12-20 10:06:23 -08:00
Chip Salzenberg
1d5e27f6d7 more error messages 2011-12-20 10:05:57 -08:00
Pieter Noordhuis
95e83386f7 Merge pull request #70 from mrb/increase_read_depth
Increase read depth by one, would allow ruby redis client to read from slowlog
2011-12-02 10:29:29 -08:00
mrb
52a29a124b increase read depth by one, would allow ruby redis client to read from slowlog 2011-12-02 08:45:13 -05:00
Pieter Noordhuis
4aa3179332 Merge pull request #67 from jzwinck/master
A small fix for outdated/typo'd comments in hiredis.c
2011-11-28 13:44:18 -08:00
John Zwinck
347528cf48 Fix minor inconsistencies in comment. 2011-11-28 16:37:35 -05:00
Pieter Noordhuis
7bc9f54134 Merge pull request #47 from geoffgarside/addrinfo
Use getaddrinfo
2011-07-20 05:02:08 -07:00
Pieter Noordhuis
6bde974927 Merge pull request #54 from rtyler/pu
Include stdlib.h in async.c for free(3), realloc(3) and strtol(3) support
2011-07-11 01:18:57 -07:00
R. Tyler Croy
e6cf455be5 Include stdlib.h in async.c for free(3), realloc(3) and strtol(3) support 2011-07-10 21:04:03 -07:00
Pieter Noordhuis
b3290392d9 Put back missing socket error check after select(2) 2011-07-10 17:51:48 +02:00
Pieter Noordhuis
b52193977e Colorize test outcome 2011-07-10 17:36:07 +02:00
Pieter Noordhuis
e40daec29a Merge branch 'printf-width' 2011-07-10 17:28:51 +02:00
Pieter Noordhuis
dad240f0a3 Test all supported types for printf-like formatting 2011-07-10 17:25:55 +02:00
Pieter Noordhuis
2da784ce8f Abort on invalid format
There is no way we can guess the width of the argument when we cannot
infer its type from the format specifier.
2011-07-09 15:08:45 +02:00
Pieter Noordhuis
27c96dde77 Use correct type when calling va_arg in formatter 2011-07-09 15:08:45 +02:00
Pieter Noordhuis
4ac55be9b5 Update printf-formatting tests to fail 2011-07-09 15:08:45 +02:00
Pieter Noordhuis
5def86434e Merge pull request #51 from rtyler/master
Minor fixes to the libev.h and other async adapters
2011-07-04 15:07:46 -07:00
Pieter Noordhuis
0625e7c5fe Update examples 2011-06-27 23:55:25 +02:00
Pieter Noordhuis
3313bcd191 Change prototype of connect callback
This commit adds a status argument to the connect callback. It will be
called in the event of an unsuccessful connection as well, where the
status argument is set to REDIS_ERR. It is set to REDIS_OK otherwise.
2011-06-27 23:52:29 +02:00
Pieter Noordhuis
e6d997a96f Extract function to check a socket for errors 2011-06-27 23:42:18 +02:00
Pieter Noordhuis
026d5ae750 Use macro's for event loop hooks 2011-06-27 22:49:25 +02:00
R. Tyler Croy
0bf4dded31 Convert the rest of the async adapters contain static functions for easier linking 2011-06-27 12:40:59 -07:00
R. Tyler Croy
62c8917f2f Make libev adapter functions static to fix linking
This will allow two different compilation units to include libev.h
and link together
2011-06-27 12:17:37 -07:00
R. Tyler Croy
0f2e899c41 Add implied include of stdlib.h 2011-06-27 12:17:37 -07:00
Pieter Noordhuis
3cc6a7f299 Bump patch version 2011-06-19 13:45:36 +02:00
Pieter Noordhuis
cecee6b764 Add changelog 2011-06-19 13:44:44 +02:00
Pieter Noordhuis
375ba48eea Revert "Add function to retrieve formatted reply"
This reverts commit 77540aa316. The change
in buffer strategy is too large to put in a minor release. It is put in
a separate branch in the meantime, so it can be refined and released
together with a minor version bump.
2011-06-19 13:41:25 +02:00
Pieter Noordhuis
f9a3229873 Fix gprof target 2011-06-19 13:13:39 +02:00
Pieter Noordhuis
54a10a27e7 Coverage report 2011-06-19 13:07:12 +02:00
Pieter Noordhuis
be5e943ddd Use CFLAGS and LDFLAGS instead of custom variables 2011-06-19 12:37:45 +02:00
Pieter Noordhuis
cc8ed7360e Append to REAL_LDFLAGS so LDFLAGS can be overridden by the user 2011-06-19 12:34:41 +02:00
Pieter Noordhuis
608e29b375 Pass LDFLAGS to linker when creating dynamic library 2011-06-19 12:33:51 +02:00
Pieter Noordhuis
ff09caa04a Remove unnecessary overrides 2011-06-19 12:16:47 +02:00
Pieter Noordhuis
888b90d6ea Don't pass CFLAGS when linking 2011-06-19 10:21:05 +02:00
Pieter Noordhuis
3331c22eb9 Add phony target 2011-06-18 17:30:59 +02:00
Pieter Noordhuis
b8e0edaabb Remove unused variables
The variable OBJARCH was used to compile objects for both 32-bit and
64-bit architectures. It can be removed because this is only relevant
for the Ruby wrapper for hiredis. This wrapper should put these flags in
CFLAGS to get the same effect.
2011-06-18 17:18:10 +02:00
Pieter Noordhuis
0c2de2525e Swap arguments 2011-06-18 17:17:48 +02:00
Pieter Noordhuis
58ebe7acbd Merge DEBUG in REAL_CFLAGS 2011-06-18 17:17:15 +02:00