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

feat: Apisix proxy cache #780

Closed
wants to merge 4 commits into from
Closed

Conversation

sjcsjc123
Copy link
Collaborator

Ⅰ. Describe what this PR did

添加proxy cache插件

Ⅱ. Does this pull request fix one issue?

fixes: #679

Ⅲ. Why don't you add test cases (unit test/integration test)?

稍后提供

Ⅳ. Describe how to verify it

补充所有测试后ci pass

Ⅴ. Special notes for reviews

有两个TODO,需要确认一下,方便进行后续开发,比较重要的点就是用户删除插件后,需要对内存或者磁盘进行一个清理

@johnlanni
Copy link
Collaborator

@sjcsjc123 可以先转为draft,后面基于redis来做吧,目前wasm里不支持磁盘操作

@sjcsjc123
Copy link
Collaborator Author

@sjcsjc123 可以先转为draft,后面基于redis来做吧,目前wasm里不支持磁盘操作

redis如何支持磁盘操作的,通过redis的持久化吗?

@sjcsjc123 sjcsjc123 marked this pull request as draft January 16, 2024 06:46
@johnlanni
Copy link
Collaborator

@sjcsjc123 可以先转为draft,后面基于redis来做吧,目前wasm里不支持磁盘操作

redis如何支持磁盘操作的,通过redis的持久化吗?

不是,就是基于redis协议做缓存,网关不关心这个缓存是否做持久化

@sjcsjc123 sjcsjc123 closed this Mar 18, 2024
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.

Support APISIX plugin - proxy-cache
2 participants