danielealbano / cachegrand

cachegrand - a modern data ingestion, processing and serving platform built for today's hardware
BSD 3-Clause "New" or "Revised" License
975 stars 34 forks source link

153 implement a code generator to to automatically create all redis commands scaffolding arguments parsing and commands callbacks #170

Closed danielealbano closed 2 years ago

danielealbano commented 2 years ago

This PR implements a new number of components with the goal to streamline the development of the Redis module and. at the same time, improve the compatibility and performances.

There are 3 new major components

This PR closes issue #153

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 5398d3673f013879523c8fec64cadcb98f0f5bd0 into 8964b241335161eb6e51d93a702d41346ca2e530 - view on LGTM.com

new alerts:

codecov[bot] commented 2 years ago

Codecov Report

Merging #170 (5521368) into main (8964b24) will increase coverage by 0.43%. The diff coverage is 77.79%.

@@            Coverage Diff             @@
##             main     #170      +/-   ##
==========================================
+ Coverage   78.04%   78.47%   +0.43%     
==========================================
  Files          95      103       +8     
  Lines        6456     6580     +124     
==========================================
+ Hits         5038     5163     +125     
+ Misses       1418     1417       -1     
Impacted Files Coverage Δ
src/config_cyaml_schema.c 100.00% <ø> (ø)
...structures/hashtable/mcmp/hashtable_support_hash.h 100.00% <ø> (ø)
...uctures/hashtable/mcmp/hashtable_support_op_arch.c 93.78% <ø> (ø)
...c/module/redis/command/module_redis_command_mset.c 0.00% <0.00%> (ø)
src/network/channel/network_channel.c 92.42% <ø> (ø)
src/network/channel/network_channel_iouring.c 100.00% <ø> (ø)
src/network/channel/network_channel_tls.c 1.24% <ø> (ø)
src/network/io/network_io_common.c 94.08% <ø> (ø)
src/network/io/network_io_common_tls.c 100.00% <ø> (ø)
src/network/network.c 80.00% <0.00%> (+2.22%) :arrow_up:
... and 36 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.