Skip to content

Speedup #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 20 commits into
base: develop
Choose a base branch
from
Draft

Speedup #9

wants to merge 20 commits into from

Conversation

JakubOnderka
Copy link
Owner

No description provided.

michael-grunder and others added 20 commits November 15, 2024 08:59
@michael-grunder
Implement KeyDB's EXPIREMEMBER[AT] commands
4cd3f59
@JakubOnderka
Initialize arrays with known size
6104b67 
We know in advance the array size, so it makes sense to avoid reallocation when adding new elements. Also use immutable empty array in when we know in advance that redis will return zero elements.
@JakubOnderka
Do not allocate empty string or string with one character
adfe23b 
From PHP 8, empty strings or string with one character don't need to be allocated. This change will reduce memory usage.
@JakubOnderka
Use immutable empty array in Redis::exec
c1afe7d
@JakubOnderka
Use immutable empty array in Redis::hKeys
b8fd8b1
@JakubOnderka
Faster parameter parsing in redis_key_cmd and redis_key_long_val_cmd
99cd89e
@JakubOnderka
Use smart str for constructing pipeline cmd
1db4414
@JakubOnderka
Optimise method array_zip_values_and_scores
0d58c39 
Initialize array with know size and reuse already allocated keys
@JakubOnderka
Initialize array with expected size for pipeline and mutli results
4fe36f7
@JakubOnderka
Use zval_get_tmp_string method that is faster when provided zval is s…
418eaa8 
…tring
@JakubOnderka
Avoid unnecessary allocation in redis_key_varval_cmd
bd77616 
This will slightly reduce memory usage for commands like RPUSH, LPUSH, SADD, SREM, etc
@JakubOnderka
Avoid unnecessary allocation in redis_hdel_cmd
05e2173 
This will slightly reduce memory usage for HDEL command
@JakubOnderka
Avoid unnecessary allocation in redis_hset_cmd
69fee35 
This will slightly reduce memory usage for HSET command
@JakubOnderka
Test for empty pipeline and multi
ed75e35
@JakubOnderka
If no command is issued in multi mode, return immutable empty array
4383fe8
@JakubOnderka
New macro to initialize packed hash array with given size
ad333b3
@JakubOnderka
Optimise HMGET method
d80165b 
Allocate output array to expected size and reuse already allocated string for output array keys
@JakubOnderka
Merge branch 'cmd-constructing-speedup' into speedup
874c624
@JakubOnderka
Merge branch 'redis_mbulk_reply_assoc' into speedup
32d7179
@JakubOnderka
Merge branch 'pipeline-cmd-smart-str' into speedup
c61ef49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JakubOnderka @michael-grunder