Skip to content
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.

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

[3.0] 新的 Redis 组件 #679

Draft
wants to merge 99 commits into
base: 3.0
Choose a base branch
from
Draft

[3.0] 新的 Redis 组件 #679

wants to merge 99 commits into from

Conversation

NHZEX
Copy link
Member

@NHZEX NHZEX commented Feb 19, 2024

重大更改检查清单:

  • 新的Lua定义方式:src/Components/queue/src/Driver/RedisQueueDriver.php
  • 新的Redis注入方式并取消大部分 use 闭包的使用方式(不存在其他IO调用且非长时间允许):src/Cache/Handler/Redis.php
  • 新的Redis驱动测试方式推广:tests/unit/Component/Tests/Cache/RedisCacheTest.php

错误修复检查清单:

推进调整清单:

  • 使用Lua类替代更多直接使用evalEx的方式
  • 依赖Redis的组件引入方式都推进为与src/Cache/Handler/Redis.php一致
  • 使用新的测试方式替换Cache*Test目前的模式(后续考虑推进到更多组件)。

@Yurunsoft
Copy link
Member

看了下大致上没有问题,需要继续完善:

  • 测试问题
  • todo 的实现
  • 文档

@NHZEX NHZEX force-pushed the 3.0-new-redis branch 14 times, most recently from ac445d3 to 211d577 Compare March 10, 2024 07:53
@NHZEX NHZEX force-pushed the 3.0-new-redis branch 3 times, most recently from 40a975c to 906e6fc Compare March 20, 2024 04:49
@NHZEX NHZEX linked an issue Mar 20, 2024 that may be closed by this pull request
@NHZEX NHZEX force-pushed the 3.0-new-redis branch 3 times, most recently from ad06eb1 to bbae0da Compare March 29, 2024 12:29
@NHZEX NHZEX force-pushed the 3.0-new-redis branch 2 times, most recently from 05f6b63 to 74bfa68 Compare April 6, 2024 14:56
# Conflicts:
#	.github/actions/ci-prepare/action.yml
#	.github/workflows/ci.yml
#	.github/workflows/phpcs.yml
#	.github/workflows/phpstan.yml
#	.github/workflows/rector.yml
#	src/Cache/Handler/Redis.php
…edis

# Conflicts:
#	.github/workflows/ci.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

支持 Redis 多种驱动,引入 predis
2 participants