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:codis-proxy bigKey/Request fuse protection #2646

Open
wants to merge 1 commit into
base: unstable
Choose a base branch
from

Conversation

XiaoLiang2333
Copy link
Contributor

@XiaoLiang2333 XiaoLiang2333 changed the title codis-proxy bigKey/Request fuse protection feat:codis-proxy bigKey/Request fuse protection May 10, 2024
@AlexStocks AlexStocks requested a review from chejinge May 10, 2024 12:24
# quick command list
quick_cmd_list = "get,set"
# slow command list
slow_cmd_list = "mget, mset"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里还是还原回去吧

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

monitor_log_max_len = 10000
# set the limitation of result set for xmonitor log
monitor_result_set_size = 20
# switch for xmonitor,0 is disabled, 1 is enabled
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里是把监控也做了吗

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

是的,除了一些特定的命令外,每一次请求/响应若被判定为bigRequest/Key、HighRisk则会存入日志,以便于使用命令"XMonitor"查看

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.

None yet

2 participants