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.

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

Add debounce mechanism for CRD event handling in CRDWatcher #24

Open
sczyh30 opened this issue Nov 10, 2022 · 1 comment
Open

Add debounce mechanism for CRD event handling in CRDWatcher #24

sczyh30 opened this issue Nov 10, 2022 · 1 comment
Assignees
@baiyutang
Labels
area/crd-watcher Category issues or PRs related to CRD watcher good first issue Good for newcomers kind/feature Category issues or PRs related to feature request.

Comments

@sczyh30
Copy link
Member

sczyh30 commented Nov 10, 2022

Add debounce mechanism for CRD event handling in CRDWatcher.

CRDWatcher 处理 CRD 变更事件时需要有 debounce 机制，避免同个 subscribeTarget(a, n, k) 短时间内多次变更导致频繁触发 push。在全量推送模型中，可以有一个简单的蓄水池机制，比如累积 100ms 的变更再合并推送。

debounce 的范围以及 timeout 需要做到可配置。欢迎社区针对这个机制进行讨论。
@sczyh30 sczyh30 added good first issue Good for newcomers kind/feature Category issues or PRs related to feature request. area/crd-watcher Category issues or PRs related to CRD watcher labels Nov 10, 2022
@baiyutang
Copy link

please assignee to me. I will finish it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@baiyutang baiyutang

Labels
area/crd-watcher Category issues or PRs related to CRD watcher good first issue Good for newcomers kind/feature Category issues or PRs related to feature request.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sczyh30 @baiyutang