Comments (4)
Hey, I put a comment up on github, but I think keeping the allocations as
powers of 2
is OK, but we need to ensure the power of two is large enough to hold what we're
going to put into it.
That is, I think the while loop is wrong, not the realloc.
Original comment by [email protected]
on 6 Apr 2009 at 6:29
from memcached.
Hi!
Thanks for a quick feedback. Either way would work fine but you've got a point,
sticking to the powers of two
approach sounds more sensible in this case. Lemmie quickly whip up another
patch and this time I'll also write a
test for it.
I'll throw another review request soon :)
Original comment by tmaesaka
on 6 Apr 2009 at 6:51
from memcached.
Here's a patch that fixes the while loop as you mentioned. Yeah, it made more
sense to fix the loop due to the
if block that comes after the loop.
http://github.com/tmaesaka/memcached/commit/e97ee810c0ee183f0a374d34e2dececdc252
77c8
I've also added a test case for this bug :)
http://github.com/tmaesaka/memcached/blob/e97ee810c0ee183f0a374d34e2dececdc25277
c8/t/issue_41.t
Cheers,
Toru
Original comment by tmaesaka
on 6 Apr 2009 at 8:10
from memcached.
Thanks so much for the test. That made a huge difference.
I want to refactor this growth thing a bit just to make it clearer. The fact
that
it's taking so long to get this right is a bad sign, it seems.
Original comment by [email protected]
on 6 Apr 2009 at 5:45
- Changed state: Fixed
from memcached.
Related Issues (20)
- SIGTERM is not handled the same as SIGINT HOT 1
- error of lru_crawler HOT 3
- lru_crawler - daemon hangs HOT 1
- internal clock drift from system time HOT 5
- Out of bounds read (Buffer overrun) HOT 1
- flags may be a negative number HOT 1
- memleaks
- why did we execute function of stop_assoc_maintenance_thread() before the process exit. HOT 2
- Cannot seem to increase the default item size above 1 MB default HOT 1
- can't start memcached in qnap ts-251 HOT 1
- Memcached cannot be controlled by start-stop-daemon HOT 1
- Cannot TAP more than 10 times (ever) HOT 1
- make test failed
- Issue with memcache services restart HOT 5
- Not Able to use "stats malloc" and "stats maps" commands in 1.4.5 HOT 1
- a low-frequency get may got wrong item HOT 5
- LRU crawler causes memcached to refuse connections on FreeBSD 10.1 HOT 2
- Memcached 1.4.23 crashes on stats command HOT 15
- memcached 1.4.23 fails test binary_prependq on Solaris 10 Sparc 64 Bit HOT 8
- GPG signature for tar.gz releases
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from memcached.