-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add micro benchmarks to fetcher.go filters. #2174
Comments
Are we primarily talking about benchmarking the |
That's correct, but maybe only for more complex ones. E.g like #2118 I no longer work on this, so feel free to grab this and finish 🤗 |
Great. I'll take it! |
Awesome. Might be good to sync with @khyatisoneji, but she is busy recently so I am pretty sure she will be happy for you to continue benchmarking. (: Feel free to checkout my branch and get all changes, then work on top of it (: |
Hey @bwplotka! I was wondering if this issue is not picked up yet, can I solve this issue? I will try and understand the requirements of this issue, before pushing in a PR! |
Of crouse, go for it @yashrsharma44 if @grzkv is not active 👍 You can see the #2118 for some start. |
@bwplotka @yashrsharma44 Sorry, I am short on time at least until the end of this week. Please feel free to pick it up. |
Hello 👋 Looks like there was no activity on this issue for last 30 days. |
Closing for now as promised, let us know if you need this to be reopened! 🤗 |
It would be awesome to learn if we need more concurrency etc there.
Those filters are critical for our components to learn about new blocks.
The text was updated successfully, but these errors were encountered: