- 16 4月, 2014 40 次提交
-
-
由 antirez 提交于
-
由 antirez 提交于
-
由 antirez 提交于
After running a few benchmarks, 3000 looks like a reasonable value to keep HLLs with a few thousand elements small while the CPU cost is still not huge. This covers all the cases where the dense representation would use N orders of magnitude more space, like in the case of many HLLs with carinality of a few tens or hundreds. It is not impossible that in the future this gets user configurable, however it is easy to pick an unreasoable value just looking at savings in the space dimension without checking what happens in the time dimension.
-
由 antirez 提交于
-
由 antirez 提交于
-
由 antirez 提交于
It is safer since it is able to have side effects.
-
由 antirez 提交于
Even if it is a debugging command, make sure that when it forces a change in encoding, the command is propagated.
-
由 antirez 提交于
If we converted to dense, a register must be updated in the dense representation.
-
由 antirez 提交于
Mostly by reordering opcodes check conditional by frequency of opcodes in larger sparse-encoded HLLs.
-
由 antirez 提交于
-
由 antirez 提交于
Bottleneck found profiling. Big run time improvement found when testing after the change.
-
由 antirez 提交于
-
由 antirez 提交于
As more values are added splitting ZERO or XZERO opcodes, try to merge adjacent VAL opcodes if they have the same value.
-
由 antirez 提交于
Now the macros will work with arguments such as "ptr+1".
-
由 antirez 提交于
-
由 antirez 提交于
Bulk length for registers was emitted too early, so if there was a bug the reply looked like a long array with just one element, blocking the client as result.
-
由 antirez 提交于
-
由 antirez 提交于
We want to promote if the total string size exceeds the resulting size after the upgrade.
-
由 antirez 提交于
The function checks if all the HLL_REGISTERS were processed during the convertion from sparse to dense encoding, returning REDIS_OK or REDIS_ERR to signal a corruption problem. A bug in PFDEBUG GETREG was fixed: when the object is converted to the dense representation we need to reassign the new pointer to the header structure pointer.
-
由 antirez 提交于
Provides a human readable description of the opcodes composing a run-length encoded HLL (sparse encoding). The command is only useful for debugging / development tasks.
-
由 antirez 提交于
PFDEBUG will be the interface to do debugging tasks with a key containing an HLL object.
-
由 antirez 提交于
The new API takes directly the object doing everything needed to turn it into a dense representation, including setting the new representation as object->ptr.
-
由 antirez 提交于
-
由 antirez 提交于
sdsIncrLen() must be called anyway even if we are replacing the last oppcode of the sparse representation.
-
由 antirez 提交于
-
由 antirez 提交于
-
由 antirez 提交于
-
由 antirez 提交于
-
由 antirez 提交于
Two vars initialized to wrong values in createHLLObject().
-
由 antirez 提交于
-
由 antirez 提交于
The function didn't considered the fact that each XZERO opcode is two bytes.
-
由 antirez 提交于
-
由 antirez 提交于
-
由 antirez 提交于
Code never tested, but the basic layout is shaped in this commit. Also missing: 1) Sparse -> Dense conversion function. 2) New HLL object creation using the sparse representation. 3) Implementation of PFMERGE for the sparse representation.
-
由 antirez 提交于
-
由 antirez 提交于
Also dense representation access macro renamed accordingly.
-
由 antirez 提交于
Metadata are now placed at the start of the representation as an header. There is a proper structure to access the representation. Still work to do in order to truly abstract the implementation from the representation, commands still work assuming dense representation.
-
由 antirez 提交于
After running a few simulations with different alternative encodings, it was found that the VAL opcode performs better using 5 bits for the value and 2 bits for the run length, at least for cardinalities in the range of interest.
-
由 antirez 提交于
-
由 antirez 提交于
adjustOpenFilesLimit() and clusterUpdateSlotsWithConfig() that were assuming uint64_t is the same as unsigned long long, which is true probably for all the systems out there that we target, but still GCC emitted a warning since technically they are two different types.
-