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

refactor: replace client side batched events by parametrizing batch_size and batch_interval #27

Closed
viniarck opened this issue Aug 31, 2023 · 1 comment · Fixed by #37
Closed

refactor: replace client side batched events by parametrizing batch_size and batch_interval #27

viniarck opened this issue Aug 31, 2023 · 1 comment · Fixed by #37
Assignees
@Ktmi
Labels
2024.1 Kytos-ng 2024.1 epic_rate_limit priority_low

Comments

@viniarck
Copy link
Member

This is for replacing client side batched events by parametrizing batch_size and batch_interval

https://github.com/kytos-ng/of_multi_table/blob/master/main.py#L330-L341

So, now clients can simply send the entire flow list and set batch_size and batch_interval accordingly to batch the flows.

This depends on kytos-ng/flow_manager#172 landing
@viniarck viniarck added 2023.2 Kytos-ng 2023.2 priority_low labels Aug 31, 2023
@viniarck
Copy link
Member Author

This will implemented with rate limiting on producer side once it's available kytos-ng/kytos#412

@viniarck viniarck added 2024.1 and removed 2023.2 Kytos-ng 2023.2 labels Feb 16, 2024
@viniarck viniarck removed the 2024.1 label Feb 27, 2024
@viniarck viniarck added 2024.1 Kytos-ng 2024.1 epic_rate_limit labels Jul 1, 2024
@Ktmi Ktmi mentioned this issue Jul 1, 2024
Merged
@viniarck viniarck assigned Ktmi and unassigned Alopalao Jul 22, 2024
@Ktmi Ktmi mentioned this issue Aug 8, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ktmi Ktmi

Labels
2024.1 Kytos-ng 2024.1 epic_rate_limit priority_low
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove Client Side Batching kytos-ng/of_multi_table
3 participants
@viniarck @Ktmi @Alopalao